Merge pull request #2018 from l2dy/carrierroute

liviuchircu · web-flow · commit 393016591c9e · 2020-03-12T13:25:32.000+01:00
carrierroute: fix return value of cr_do_route
diff --git a/modules/carrierroute/route_func.c b/modules/carrierroute/route_func.c
@@ -568,8 +568,9 @@ int cr_do_route(struct sip_msg * _msg, void *_carrier,
 	if (set_ruri(_msg, &dest) < 0) {
 		LM_ERR("Error setting RURI\n");
 		ret = -1;
+	} else {
+		ret = 1;
 	}
-	ret = 1;
 
 	pkg_free(dest.s);
 

Original file line number	Diff line number	Diff line change
`@@ -568,8 +568,9 @@ int cr_do_route(struct sip_msg * _msg, void *_carrier,`
`568`	`568`	`if (set_ruri(_msg, &dest) < 0) {`
`569`	`569`	`LM_ERR("Error setting RURI\n");`
`570`	`570`	`ret = -1;`
	`571`	`+ } else {`
	`572`	`+ ret = 1;`
`571`	`573`	`}`
`572`		`- ret = 1;`
`573`	`574`
`574`	`575`	`pkg_free(dest.s);`
`575`	`576`