Andrew's git
/
gitweb.git
/ diff
summary
|
log
|
commit
| diff |
tree
commit
grep
author
committer
pickaxe
?
re
Revert "run-command: prettify -D_FORTIFY_SOURCE workaround"
author
Junio C Hamano
<gitster@pobox.com>
Mon, 18 Apr 2011 21:14:53 +0000
(14:14 -0700)
committer
Junio C Hamano
<gitster@pobox.com>
Mon, 18 Apr 2011 21:14:53 +0000
(14:14 -0700)
This reverts commit
ebec842773932e6f853acac70c80f84209b5f83e
, which
somehow mistakenly thought that any non-zero return from write(2) is
an error.
run-command.c
patch
|
blob
|
history
raw
|
patch
|
inline
| side by side (parent:
4d8b32a
)
diff --git
a/run-command.c
b/run-command.c
index 8619c769a93c48e724957c019af5765ebee6af9f..f91e446c86be8e27f98554567143d7ce6f934bd1 100644
(file)
--- a/
run-command.c
+++ b/
run-command.c
@@
-67,26
+67,21
@@
static int child_notifier = -1;
static void notify_parent(void)
{
static void notify_parent(void)
{
- /*
- * execvp failed. If possible, we'd like to let start_command
- * know, so failures like ENOENT can be handled right away; but
- * otherwise, finish_command will still report the error.
- */
- if (write(child_notifier, "", 1))
- ; /* yes, dear gcc -D_FORTIFY_SOURCE, there was an error. */
+ ssize_t unused;
+ unused = write(child_notifier, "", 1);
}
static NORETURN void die_child(const char *err, va_list params)
{
char msg[4096];
}
static NORETURN void die_child(const char *err, va_list params)
{
char msg[4096];
+ ssize_t unused;
int len = vsnprintf(msg, sizeof(msg), err, params);
if (len > sizeof(msg))
len = sizeof(msg);
int len = vsnprintf(msg, sizeof(msg), err, params);
if (len > sizeof(msg))
len = sizeof(msg);
- if (write(child_err, "fatal: ", 7) ||
- write(child_err, msg, len) ||
- write(child_err, "\n", 1))
- ; /* yes, gcc -D_FORTIFY_SOURCE, we know there was an error. */
+ unused = write(child_err, "fatal: ", 7);
+ unused = write(child_err, msg, len);
+ unused = write(child_err, "\n", 1);
exit(128);
}
#endif
exit(128);
}
#endif