[PATCH] fix git-checkout-cache segfault on parisc
authorJames Bottomley <James.Bottomley@SteelEye.com>
Sat, 13 Aug 2005 15:29:32 +0000 (10:29 -0500)
committerJunio C Hamano <junkio@cox.net>
Sun, 14 Aug 2005 01:28:52 +0000 (18:28 -0700)
The index cleanup code is executed via atexit() which is *after* main
has completed, so the stack allocated cache_file has gone out of scope.
Parisc seems to use stack in the destructor functions, so cache_file
gets partially overwritten leading to the predictable disastrous
consequences.

[jc: Just to make sure, I audited other users of the function
 hold_index_file_for_update() to make sure they do not have this
 problem; everybody else uses non-stack cache_file structure and
 is fine.  Thanks, James.]

Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Signed-off-by: Junio C Hamano <junkio@cox.net>
checkout-cache.c

index 8d67403ae86c6deafe4c2db5fef8af64f9fe6ecc..31ff577500bf07ec67a16ac00b91808bfa913c12 100644 (file)
@@ -78,10 +78,11 @@ static int checkout_all(void)
 static const char checkout_cache_usage[] =
 "git-checkout-cache [-u] [-q] [-a] [-f] [-n] [--prefix=<string>] [--] <file>...";
 
+static struct cache_file cache_file;
+
 int main(int argc, char **argv)
 {
        int i, force_filename = 0;
-       struct cache_file cache_file;
        int newfd = -1;
 
        if (read_cache() < 0) {