Thread (7 messages) flat view 7 messages, 3 authors, 2016-06-15

Re: 'git fast-export' is crashing on the gcc repo

From: Alex Riesen <hidden>
Date: 2016-06-15 22:43:58

Nicolas Pitre, Tue, Dec 11, 2007 23:06:42 +0100:
Well, ignore the above.  It seems that most of stdio doesn't set errno 
so the above is crap.
Well, it had no reason to in this case. It's not an error.
It does not even have to do a syscall.
quoted hunk ↗ jump to hunk
diff --git a/builtin-fast-export.c b/builtin-fast-export.c
index 2136aad..c32a124 100755
--- a/builtin-fast-export.c
+++ b/builtin-fast-export.c
@@ -103,7 +103,7 @@ static void handle_object(const unsigned char *sha1)
 	mark_object(object);
 
 	printf("blob\nmark :%d\ndata %lu\n", last_idnum, size);
-	if (fwrite(buf, size, 1, stdout) != 1)
+	if (fwrite(buf, 1, size, stdout) != size)
That's a probable syscall which could be spared
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help