On Feb 2, 2011, at 4:51 PM, Bogdan-Andrei Iancu wrote:
> Hi Brett,
>
> The error log you get is triggered when the internal function "next_branches"
> returns false (a negative code). This is when:
> - function called from wrong route type (like for a reply)
> - there are no more branches left
> - internal error
>
> I guess in your case, the second case (when there are no more branches) is
> the trigger. And indeed, in this case, the error message is bogus - we need
> to fix that.
Here's one possible solution for consideration.
diff --git a/serialize.c b/serialize.c
index 7b16055..837c321 100644
--- a/serialize.c
+++ b/serialize.c
@@ -273,7 +273,7 @@ int next_branches( struct sip_msg *msg)
if (!avp) {
LM_DBG("no AVPs -- we are done!\n");
- goto error;
+ return 2;
}
if (!val.s.s) {
_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users
