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 93ffc4ab930944511c96d5b0936a9d8f)

Currently displaying revision 93ffc4ab930944511c96d5b0936a9d8f , 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 93ffc4ab930944511c96d5b0936a9d8f)

Places