aboutsummaryrefslogtreecommitdiff
path: root/builtin-add.c
diff options
context:
space:
mode:
authorJim Meyering <meyering@redhat.com>2009-09-12 10:43:27 +0200
committerJunio C Hamano <gitster@pobox.com>2009-09-13 01:32:20 -0700
commit41698375ada57d51cf6db5cf25970ccd1b918180 (patch)
treea1ed2b0f32a829bdb0b4cece3d85df933b924962 /builtin-add.c
parent3d913526b19545eb4e463f96d2f3b8db8daae6ef (diff)
downloadgit-41698375ada57d51cf6db5cf25970ccd1b918180.tar.gz
git-41698375ada57d51cf6db5cf25970ccd1b918180.tar.xz
don't dereference NULL upon fdopen failure
There were several unchecked use of fdopen(); replace them with xfdopen() that checks and dies. Signed-off-by: Jim Meyering <meyering@redhat.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin-add.c')
-rw-r--r--builtin-add.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/builtin-add.c b/builtin-add.c
index a5714735e..cb6e5906f 100644
--- a/builtin-add.c
+++ b/builtin-add.c
@@ -198,7 +198,7 @@ static int edit_patch(int argc, const char **argv, const char *prefix)
out = open(file, O_CREAT | O_WRONLY, 0644);
if (out < 0)
die ("Could not open '%s' for writing.", file);
- rev.diffopt.file = fdopen(out, "w");
+ rev.diffopt.file = xfdopen(out, "w");
rev.diffopt.close_file = 1;
if (run_diff_files(&rev, 0))
die ("Could not write patch");