In very obscure situations we may write the free token back to the pipe.

Don't do that.  I couldn't come up with a repro case for this!
This commit is contained in:
Paul Smith 2011-11-14 00:18:38 +00:00
parent bfc3e1ca7c
commit 934f51d166
2 changed files with 6 additions and 2 deletions

View File

@ -1,8 +1,11 @@
2011-11-13 Paul Smith <psmith@gnu.org> 2011-11-13 Paul Smith <psmith@gnu.org>
* main.c (clean_jobserver): Don't write the free token to the pipe.
Change suggested by Tim Newsome <tnewsome@aristanetworks.com>
* acinclude.m4 (AC_STRUCT_ST_MTIM_NSEC): Add support for Darwin. * acinclude.m4 (AC_STRUCT_ST_MTIM_NSEC): Add support for Darwin.
Patch provided by Troy Runkel <Troy.Runkel@mathworks.com>
* filedef.h (FILE_TIMESTAMP_STAT_MODTIME): Ditto. * filedef.h (FILE_TIMESTAMP_STAT_MODTIME): Ditto.
Patch provided by Troy Runkel <Troy.Runkel@mathworks.com>
2011-09-18 Paul Smith <psmith@gnu.org> 2011-09-18 Paul Smith <psmith@gnu.org>

3
main.c
View File

@ -3116,7 +3116,8 @@ clean_jobserver (int status)
"INTERNAL: Exiting with %u jobserver tokens (should be 0)!", "INTERNAL: Exiting with %u jobserver tokens (should be 0)!",
jobserver_tokens); jobserver_tokens);
else else
while (jobserver_tokens--) /* Don't write back the "free" token */
while (--jobserver_tokens)
{ {
int r; int r;