@asalman18 commented on this pull request.
@@ -2270,6 +2372,15 @@ int ds_manage_routes(sip_msg_t *msg, ds_select_state_t *rstate)
case DS_ALG_PARALLEL: /* 12 - parallel dispatching */ hash = 0; break; + case DS_ALG_LATENCY: /* 13 - latency optimized round-robin with failover */ + lock_get(&idx->lock); + hash = ds_manage_route_algo13(idx, rstate); + lock_release(&idx->lock); + if (hash < 0)
Should we check if hash is -1 and not < 0