summaryrefslogtreecommitdiff
path: root/upload-pack.c
diff options
context:
space:
mode:
authorJim Meyering <meyering@redhat.com>2009-09-12 08:43:27 (GMT)
committerJunio C Hamano <gitster@pobox.com>2009-09-13 08:32:20 (GMT)
commit41698375ada57d51cf6db5cf25970ccd1b918180 (patch)
treea1ed2b0f32a829bdb0b4cece3d85df933b924962 /upload-pack.c
parent3d913526b19545eb4e463f96d2f3b8db8daae6ef (diff)
downloadgit-41698375ada57d51cf6db5cf25970ccd1b918180.zip
git-41698375ada57d51cf6db5cf25970ccd1b918180.tar.gz
git-41698375ada57d51cf6db5cf25970ccd1b918180.tar.bz2
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 'upload-pack.c')
-rw-r--r--upload-pack.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/upload-pack.c b/upload-pack.c
index b3471e4..38ddac2 100644
--- a/upload-pack.c
+++ b/upload-pack.c
@@ -108,7 +108,7 @@ static int do_rev_list(int fd, void *create_full_pack)
int i;
struct rev_info revs;
- pack_pipe = fdopen(fd, "w");
+ pack_pipe = xfdopen(fd, "w");
init_revisions(&revs, NULL);
revs.tag_objects = 1;
revs.tree_objects = 1;
@@ -255,7 +255,7 @@ static void create_pack_file(void)
/* pass on revisions we (don't) want */
if (!shallow_nr) {
- FILE *pipe_fd = fdopen(pack_objects.in, "w");
+ FILE *pipe_fd = xfdopen(pack_objects.in, "w");
if (!create_full_pack) {
int i;
for (i = 0; i < want_obj.nr; i++)