Hi Jeff,

Try this patch to get the callid printed out.

Regards,
Bogdan

Jeff Pyle wrote:
Bogdan,

I'm not sure if I have a trace or not.  A lot of calls flow through this switch 
and this error only appears for a very small percentage of them.  Is it 
possible to include the callid in the error so I can reference it to siptrace 
logs?

- Jeff


On Feb 17, 2010, at 3:57 AM, Bogdan-Andrei Iancu wrote:

Hi Jeff,

The error message may be a bit misleading - it is not about new uri shorter then old uri, is about the URI from Route hdr being shorter than the old URI.

The URI encoded in the Route param has the len = max(new uri, old uri), so the module expects that the URI from FROM to be equal of shorter than the one from Route param.

Do you have a trace of the bogus call ? or at least of the message ? I suspect some of the phones do mess with the Route param.

Regards,
Bogdan

Jeff Pyle wrote:
Hello,

I make use of uac_replace_from() in my scripts.  I believe that is where this 
error message is coming from:
 ERROR:uac:restore_uri: new URI shorter than old URI

Can anyone explain what this really means?  Having a new URI shorter than an 
old one really doesn't seem like an error condition to me.

I haven't changed my script in months.  This error seems to have surfaced 
somewhere around 1.6's 6590 build.


Thanks,
Jeff


_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users


--
Bogdan-Andrei Iancu
www.voice-system.ro


_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users


_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users



--
Bogdan-Andrei Iancu
www.voice-system.ro

Index: modules/uac/replace.c
===================================================================
--- modules/uac/replace.c       (revision 6608)
+++ modules/uac/replace.c       (working copy)
@@ -458,7 +458,12 @@
 
        /* get new uri */
        if ( new_uri.len<old_uri.len ) {
-               LM_ERR("new URI shorter than old URI\n");
+               parse_headers(msg,HDR_CALLID_F,0);
+               if (msg->callid)
+                       LM_ERR("new URI shorter than old URI (callid=%.*s)\n",
+                               msg->callid->body.len,msg->callid->body.s);
+               else
+                       LM_ERR("new URI shorter than old URI (callid=?)\n");
                goto failed;
        }
        for( i=0 ; i<old_uri.len ; i++ )
_______________________________________________
Users mailing list
[email protected]
http://lists.opensips.org/cgi-bin/mailman/listinfo/users

Reply via email to