Patch review for fix to #947 (assert on SIGHUP to varnishncsa)

Martin Blix Grydeland martin at varnish-software.com
Fri Jul 8 16:43:24 CEST 2011


This bug is caused by an assert when usleep is interrupted by signal
delivery in vsl.c (libvarnishapi). Some tools expect to receive signals
(e.g. varnishncsa with -w option), so this has to be addressed. Attached
patch relaxes the assert to not trigger on signal interrupt. Any comments
appreciated.

Regards,
Martin Blix Grydeland

-- 
Martin Blix Grydeland
Varnish Software AS
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20110708/c6d7e9a3/attachment-0003.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug-942-assert-on-sighup.patch
Type: application/octet-stream
Size: 732 bytes
Desc: not available
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-dev/attachments/20110708/c6d7e9a3/attachment-0003.obj>


More information about the varnish-dev mailing list