diff options
author | Eric Wong <normalperson@yhbt.net> | 2009-05-28 10:35:11 -0700 |
---|---|---|
committer | Eric Wong <normalperson@yhbt.net> | 2009-05-28 11:45:15 -0700 |
commit | 894631ab86de364fda9a12693896678f1c9795f8 (patch) | |
tree | c33e84f491040dde0af4f03693b01015708d208a /SIGNALS | |
parent | 59c3e80ceb6cff688b8bf3ed7dfc00ff041fca48 (diff) | |
download | unicorn-894631ab86de364fda9a12693896678f1c9795f8.tar.gz |
There is a potential race condition in closing the tempfile immediately after SIGKILL-ing the child. So instead just close it when we reap the dead child. Some some versions of Ruby may also not like having the WORKERS hash being changed while it is iterating through each_pair, so dup the hash to be on the safe side (should be cheap, since it's not a deep copy) since kill_worker() can modify the WORKERS hash. This is somewhat of a shotgun fix as I can't reproduce the problem consistently, but oh well.
Diffstat (limited to 'SIGNALS')
0 files changed, 0 insertions, 0 deletions