Changeset 1238:8f9684264fe0 in freeDiameter for libfdcore/p_dp.c

Timestamp:

Oct 10, 2013, 11:08:46 PM (11 years ago)

Author:

Sebastien Decugis <sdecugis@freediameter.net>

Branch:

default

Phase:

public

Message:

Change management of the p_reqin_count counter to be updated only on routable messages. This should limit the errors in the counter value resulting from rejected or discarded link-local messages.

File:

• libfdcore/p_dp.c (modified) (3 diffs)

libfdcore/p_dp.c

@@ -113 +113 @@
                 CHECK_FCT( fd_peer_get_load_pending(&peer->p_hdr, &to_receive, &to_send) );
                 
-                if ((to_receive == 0) && (to_send == 1 /* only the DPA */)) {
+                if ((to_receive == 0) && (to_send == 0)) {
                         /* No pending exchange, move to CLOSING directly */
                         CHECK_FCT( fd_psm_change_state(peer, STATE_CLOSING) );
                 
                         /* Now send the DPA */
-                        CHECK_FCT( fd_out_send( msg, NULL, peer) );
+                        CHECK_FCT( fd_out_send( msg, NULL, peer, 0) );
                         
                         /* and move to CLOSED */
@@ -132 +132 @@
                         
                         /* Now send the DPA */
-                        CHECK_FCT( fd_out_send( msg, NULL, peer) );
+                        CHECK_FCT( fd_out_send( msg, NULL, peer, 0) );
                 }
         } else {
@@ -202 +202 @@
         
         /* Now send the DPR message */
-        CHECK_FCT_DO( fd_out_send(&msg, NULL, peer), /* ignore since we are on timeout anyway */ );
+        CHECK_FCT_DO( fd_out_send(&msg, NULL, peer, 0), /* ignore since we are on timeout anyway */ );
         
         return 0;