From: Jim Meyering Date: Wed, 27 Jun 2007 14:28:53 +0000 (+0200) Subject: git-log: detect dup and fdopen failure X-Git-Tag: v1.5.3-rc0~47 X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=f57882505efe05990102a0d96b37c09baadae03d;p=git.git git-log: detect dup and fdopen failure This defines xdup() and xfdopen() in git-compat-util.h to give us error-catching variants of them without cluttering the code too much. Signed-off-by: Jim Meyering Acked-by: Linus Torvalds Signed-off-by: Junio C Hamano --- diff --git a/builtin-log.c b/builtin-log.c index 073a2a16a..a4186eac8 100644 --- a/builtin-log.c +++ b/builtin-log.c @@ -589,7 +589,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix) get_patch_ids(&rev, &ids, prefix); if (!use_stdout) - realstdout = fdopen(dup(1), "w"); + realstdout = xfdopen(xdup(1), "w"); prepare_revision_walk(&rev); while ((commit = get_revision(&rev)) != NULL) { diff --git a/git-compat-util.h b/git-compat-util.h index b2ab3f825..362e040f5 100644 --- a/git-compat-util.h +++ b/git-compat-util.h @@ -287,6 +287,22 @@ static inline ssize_t xwrite(int fd, const void *buf, size_t len) } } +static inline int xdup(int fd) +{ + int ret = dup(fd); + if (ret < 0) + die("dup failed: %s", strerror(errno)); + return ret; +} + +static inline FILE *xfdopen(int fd, const char *mode) +{ + FILE *stream = fdopen(fd, mode); + if (stream == NULL) + die("Out of memory? fdopen failed: %s", strerror(errno)); + return stream; +} + static inline size_t xsize_t(off_t len) { return (size_t)len;