[sr-dev] git:3.1: modules/carrieroute: fixed a bug in carrieroute

Module: sip-router Branch: 3.1 Commit: ea8430ddb32e2f45f29362a97ffed2fee011f6a3 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ea8430d… Author: Marius Bucur &lt;marius.bucur(a)1and1.ro&gt; Committer: Marius Bucur &lt;marius.bucur(a)1and1.ro&gt; Date: Wed May 11 13:43:08 2011 +0300 modules/carrieroute: fixed a bug in carrieroute the bug caused carrieroute to crash when an invalid SIP message is received (e.g. with no callid) --- modules/carrierroute/cr_func.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/carrierroute/cr_func.c b/modules/carrierroute/cr_func.c index 714a130..1006228 100644 --- a/modules/carrierroute/cr_func.c +++ b/modules/carrierroute/cr_func.c @@ -437,7 +437,7 @@ static int rewrite_on_rule(struct route_flags *rf_head, flag_t flags, str * dest } break; case alg_crc32_nofallback: - if ((prob = (hash_func(msg, hash_source, rf->max_targets) + 1)) < 0) { + if ((prob = (hash_func(msg, hash_source, rf->max_targets))) < 0) { LM_ERR("could not hash message with CRC32"); return -1; } @@ -445,7 +445,7 @@ static int rewrite_on_rule(struct route_flags *rf_head, flag_t flags, str * dest * this function just tries only a backup rule and otherwise * returns -1. This way we get an error */ - if ((rr = get_rule_by_hash(rf, prob)) == NULL) { + if ((rr = get_rule_by_hash(rf, prob + 1)) == NULL) { LM_CRIT("no route found\n"); return -1; }