diff options
| author | David Goulet <dgoulet@ev0ke.net> | 2015-01-30 14:59:48 -0500 |
|---|---|---|
| committer | David Goulet <dgoulet@ev0ke.net> | 2015-01-30 15:05:18 -0500 |
| commit | 51f793e37e55b29700f76ca9cdb3199e636bb51d (patch) | |
| tree | e8d628a7662d51a41258a5215035f0dd5c26873f /changes | |
| parent | 44e9dafb67370aa6fb163cda150a80466f668111 (diff) | |
| download | tor-51f793e37e55b29700f76ca9cdb3199e636bb51d.tar.gz tor-51f793e37e55b29700f76ca9cdb3199e636bb51d.zip | |
Fix possible infinite loop on pipe/sock_drain()
If the returned value of read/recv is 0 (meaning EOF), we'll end up in an
infinite loop (active wait) until something is written on the pipe which is
not really what we want here especially because those functions are called
from the main thread.
Signed-off-by: David Goulet <dgoulet@ev0ke.net>
Diffstat (limited to 'changes')
| -rw-r--r-- | changes/bug14554 | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/changes/bug14554 b/changes/bug14554 new file mode 100644 index 0000000000..ff2566f7af --- /dev/null +++ b/changes/bug14554 @@ -0,0 +1,4 @@ + o Major bugfix + - Possibility of an infinite loop if the returned value of the read/recv + was 0. A returned value of 0 means that we've reached the EOF thus the + pipe/sock is drained so return success not an error. |