[PATCH] Remove sys_poll() (raceless signal handler)?
Andrew Bartlett
abartlet at samba.org
Wed Feb 8 22:14:12 MST 2012
This patch comes out of some work I've been doing with callcatcher, to
find dead code in Samba.
It seems we do not need the sys_poll() wrapper, as something else
(tevent presumably) handles the creation of the signal file descriptor.
Either that, or we have a problem with the 'race-less signal handler',
as none of the signal handlers call the magic sys_select_signal()
function.
I'm still unclear why we call poll directly, particularly in
smbd/process.c, hence why I post these patches for review, but more
particularly for comment.
Thanks,
Andrew Bartlett
--
Andrew Bartlett http://samba.org/~abartlet/
Authentication Developer, Samba Team http://samba.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lib-util-Remove-unused-sys_select_signal.patch
Type: text/x-patch
Size: 1620 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20120209/34ec9973/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-lib-util-Remove-sys_poll-as-it-is-no-longer-needed.patch
Type: text/x-patch
Size: 8468 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20120209/34ec9973/attachment-0001.bin>
More information about the samba-technical
mailing list