Changeset 166:6d166f75e25a in freeDiameter

Timestamp:

Jan 28, 2010, 3:01:50 PM (14 years ago)

Author:

Sebastien Decugis <sdecugis@nict.go.jp>

Branch:

default

Phase:

public

Message:

Fix routing list scores initialization

Files:

• extensions/dbg_rt/dbg_rt.c (modified) (1 diff)
• libfreeDiameter/rt_data.c (modified) (2 diffs)

extensions/dbg_rt/dbg_rt.c

@@ -64 +64 @@
         fd_log_debug("[dbg_rt] OUT routing message: %p\n", msg);
         fd_msg_dump_walk(INFO, msg);
-        fd_log_debug("[dbg_rt] Current list of candidates (%p)\n", msg);
+        fd_log_debug("[dbg_rt] Current list of candidates (%p): (score - id)\n", msg);
         
         for (li = candidates->next; li != candidates; li = li->next) {
                 struct rtd_candidate *c = (struct rtd_candidate *) li;
-                fd_log_debug("[dbg_rt]   - %d\t%s\n", c->score, c->diamid);
+                fd_log_debug("[dbg_rt]   %d -\t%s\n", c->score, c->diamid);
         }
         

libfreeDiameter/rt_data.c

@@ -226 +226 @@
 }
 
-/* Extract the list of valid candidates, and initialize their scores to 0 */
+/* Extract the list of valid candidates, and initialize their scores */
 void fd_rtd_candidate_extract(struct rt_data * rtd, struct fd_list ** candidates, int ini_score)
 {
+        struct fd_list * li;
+        
         TRACE_ENTRY("%p %p", rtd, candidates);
         CHECK_PARAMS_DO( candidates, return );
@@ -235 +237 @@
         *candidates = &rtd->candidates;
         
-        if (rtd->extracted) {
-                /* Reset all scores to INITIAL score */
-                struct fd_list * li;
-                for (li = rtd->candidates.next; li != &rtd->candidates; li = li->next) {
-                        struct rtd_candidate * c = (struct rtd_candidate *) li;
-                        c->score = ini_score;
-                }
+        /* Reset all scores to INITIAL score */
+        for (li = rtd->candidates.next; li != &rtd->candidates; li = li->next) {
+                struct rtd_candidate * c = (struct rtd_candidate *) li;
+                c->score = ini_score;
         }