Changeset 691:78b665400097 in freeDiameter for libfdcore/p_sr.c

Timestamp:

Jan 20, 2011, 7:44:27 PM (13 years ago)

Author:

Sebastien Decugis <sdecugis@nict.go.jp>

Branch:

default

Phase:

public

Message:

Cleanup all pthread_cleanup_push / pop pairs so that pop is always called after push, or ASSERT(0) is some grave errors

File:

• libfdcore/p_sr.c (modified) (2 diffs)

libfdcore/p_sr.c

@@ -117 +117 @@
 }
 
-/* thread that handles messages expiring. The thread is started / stopped only when needed */
+/* thread that handles messages expiring. The thread is started / cancelled only when needed */
 static void * sr_expiry_th(void * arg) {
         struct sr_list * srlist = arg;
@@ -184 +184 @@
         ; /* pthread_cleanup_pop sometimes expands as "} ..." and the label beofre this cause some compilers to complain... */
         pthread_cleanup_pop( 1 );
+        ASSERT(0); /* we have encountered a problem, maybe time to signal the framework to terminate? */
         return NULL;
 }