Code

fetch-pack: do not use lockfile structure on stack.
authorJunio C Hamano <junkio@cox.net>
Tue, 2 Jan 2007 19:22:08 +0000 (11:22 -0800)
committerJunio C Hamano <junkio@cox.net>
Tue, 2 Jan 2007 19:22:08 +0000 (11:22 -0800)
They are used in atexit() for clean-up, and you will be
accessing unallocated memory at that point.

See 31f584c2 for the fix for a similar problem.

Signed-off-by: Junio C Hamano <junkio@cox.net>
fetch-pack.c

index c527bf9e9621519f72e1039313666a03ee12c586..1530a947947b68d879e01bb9161654e135716a3d 100644 (file)
@@ -625,6 +625,8 @@ static int remove_duplicates(int nr_heads, char **heads)
        return dst;
 }
 
+static struct lock_file lock;
+
 int main(int argc, char **argv)
 {
        int i, ret, nr_heads;
@@ -632,7 +634,6 @@ int main(int argc, char **argv)
        int fd[2];
        pid_t pid;
        struct stat st;
-       struct lock_file lock;
 
        setup_git_directory();