File unix_thread-Remove-the-fd-handler-sets-before-i... of Package OpenIPMI

Overview Repositories Revisions Requests Users Attributes Meta

File unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch of Package OpenIPMI (Revision c7cfc145e1ee87e8d43ed4c668a55245)

Currently displaying revision c7cfc145e1ee87e8d43ed4c668a55245 , Show latest

From: Corey Minyard <cminyard@mvista.com>
Subject: unix_thread: Remove the fd handler sets before it's set up
References: 
Patch-Mainline: 
Git-commit: d613d279dbce2d5e4594f6fed39653d83af0d99b
Git-repo: https://git.code.sf.net/p/openipmi/code.git

There were a couple of fd handler set before the fd was actually
configured.  With recent changes in the selector code that would result
in an assert.  The calls were unnecessary, so remove them.

Signed-off-by: Corey Minyard <cminyard@mvista.com>

Signed-off-by:  <trenn@suse.com>
diff --git a/unix/posix_thread_os_hnd.c b/unix/posix_thread_os_hnd.c
index 28c66898..cb315c5e 100644
--- a/unix/posix_thread_os_hnd.c
+++ b/unix/posix_thread_os_hnd.c
@@ -140,8 +140,6 @@ add_fd(os_handler_t       *handler,
     fd_data->data_ready = data_ready;
     fd_data->handler = handler;
     fd_data->freed = freed;
-    sel_set_fd_write_handler(posix_sel, fd, SEL_FD_HANDLER_DISABLED);
-    sel_set_fd_except_handler(posix_sel, fd, SEL_FD_HANDLER_DISABLED);
     rv = sel_set_fd_handlers(posix_sel, fd, fd_data, fd_handler, NULL, NULL,
 			     free_fd_data);
     if (rv) {

Places

File unix_thread-Remove-the-fd-handler-sets-before-it-s-set-up.patch of Package OpenIPMI (Revision c7cfc145e1ee87e8d43ed4c668a55245)

Places