Changeset 455:7250e91f0662 in freeDiameter for freeDiameter/p_out.c

Timestamp:

Jul 29, 2010, 4:42:29 PM (14 years ago)

Author:

Sebastien Decugis <sdecugis@nict.go.jp>

Branch:

default

Phase:

public

Message:

Some additional flush enforcement to attempt to avoid some possible problems

File:

• freeDiameter/p_out.c (modified) (2 diffs)

freeDiameter/p_out.c

@@ -122 +122 @@
                 CHECK_FCT_DO( do_send(&msg, 0, peer->p_cnxctx, &peer->p_hbh, &peer->p_sr),
                         {
-                                fd_log_debug("An error occurred while sending this message, it is lost:\n");
-                                fd_msg_dump_walk(NONE, msg);
-                                fd_msg_free(msg);
+                                if (msg) {
+                                        fd_log_debug("An error occurred while sending this message, it was lost:\n");
+                                        fd_msg_dump_walk(NONE, msg);
+                                        fd_msg_free(msg);
+                                }
                         } );
                         
@@ -161 +163 @@
                 CHECK_FCT_DO( do_send(msg, flags, cnx, hbh, peer ? &peer->p_sr : NULL),
                         {
-                                fd_log_debug("An error occurred while sending this message, it is lost:\n");
-                                fd_msg_dump_walk(NONE, *msg);
-                                fd_msg_free(*msg);
-                                *msg = NULL;
+                                if (msg) {
+                                        fd_log_debug("An error occurred while sending this message, it was lost:\n");
+                                        fd_msg_dump_walk(NONE, *msg);
+                                        fd_msg_free(*msg);
+                                        *msg = NULL;
+                                }
                         } );
         }