Changeset 427 for vendor/current/lib/tevent

Timestamp:

Apr 9, 2010, 3:20:58 PM (15 years ago)

Author:

Silvan Scherrer

Message:

Samba 3.5.x: update to 3.5.2

Location:

vendor/current/lib/tevent

Files:

• tevent.c (modified) (1 diff)
• tevent_internal.h (modified) (1 diff)
• tevent_signal.c (modified) (3 diffs)

vendor/current/lib/tevent/tevent.c

@@ -177 +177 @@
                 se->event_ctx = NULL;
                 DLIST_REMOVE(ev->signal_events, se);
+                /*
+                 * This is important, Otherwise signals
+                 * are handled twice in child. eg, SIGHUP.
+                 * one added in parent, and another one in
+                 * the child. -- BoYang
+                 */
+                tevent_cleanup_pending_signal_handlers(se);
         }
 

vendor/current/lib/tevent/tevent_internal.h

@@ -300 +300 @@
                                                const char *location);
 int tevent_common_check_signal(struct tevent_context *ev);
+void tevent_cleanup_pending_signal_handlers(struct tevent_signal *se);
 
 bool tevent_standard_init(void);

vendor/current/lib/tevent/tevent_signal.c

@@ -134 +134 @@
 static int tevent_common_signal_list_destructor(struct tevent_common_signal_list *sl)
 {
-        DLIST_REMOVE(sig_state->sig_handlers[sl->se->signum], sl);
+        if (sig_state->sig_handlers[sl->se->signum]) {
+                DLIST_REMOVE(sig_state->sig_handlers[sl->se->signum], sl);
+        }
         return 0;
 }
@@ -155 +157 @@
         if (sig_state->sig_handlers[se->signum] == NULL) {
                 /* restore old handler, if any */
-                sigaction(se->signum, sig_state->oldact[se->signum], NULL);
-                sig_state->oldact[se->signum] = NULL;
+                if (sig_state->oldact[se->signum]) {
+                        sigaction(se->signum, sig_state->oldact[se->signum], NULL);
+                        sig_state->oldact[se->signum] = NULL;
+                }
 #ifdef SA_SIGINFO
                 if (se->sa_flags & SA_SIGINFO) {
-                        talloc_free(sig_state->sig_info[se->signum]);
-                        sig_state->sig_info[se->signum] = NULL;
+                        if (sig_state->sig_info[se->signum]) {
+                                talloc_free(sig_state->sig_info[se->signum]);
+                                sig_state->sig_info[se->signum] = NULL;
+                        }
                 }
 #endif
@@ -399 +405 @@
         return 1;
 }
+
+void tevent_cleanup_pending_signal_handlers(struct tevent_signal *se)
+{
+        struct tevent_common_signal_list *sl;
+        sl = talloc_get_type(se->additional_data,
+                             struct tevent_common_signal_list);
+
+        tevent_common_signal_list_destructor(sl);
+
+        if (sig_state->sig_handlers[se->signum] == NULL) {
+                if (sig_state->oldact[se->signum]) {
+                        sigaction(se->signum, sig_state->oldact[se->signum], NULL);
+                        sig_state->oldact[se->signum] = NULL;
+                }
+        }
+        return;
+}