Negative return code from dispatcher algo route will cause the entry …

…to be automatically skipped

modules/dispatcher/dispatch.c

@@ -808,6 +808,12 @@ int ds_route_algo(struct sip_msg *msg, ds_set_p set,
 		fret = run_route_algo(msg, algo_route->idx, &set->dlist[i]);
 		set->dlist[i].route_algo_value = fret;
 
+		if (fret < 0) {
+			/* move to the end of the list */
+			sset[end_idx--] = &set->dlist[i];
+			continue;
+		}
+
 		/* search the proper position */
 		j = 0;
 		for (; j < cnt && sset[j]->route_algo_value <= fret; j++);

modules/dispatcher/doc/dispatcher_admin.xml

@@ -184,6 +184,8 @@ modparam("dispatcher", "attrs_avp", "$avp(script_attrs)")
 		The return value of the route is considered by the dispatcher module to
 		be the current weight of the dispatcher entry, and when using the 10
 		algo, the dispatcher entries are sorted in ascending weight order.
+
+		If the returned value from the algo route is negative, the current dispatcher entry will be automatically skipped from usage
 		</para>
 		<para>
 		<emphasis>