Bugs item #2204065, was opened at 2008-10-28 15:06
Message generated for change (Comment added) made by bogdan_iancu
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=2204065&group_id=232389
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: core
Group: trunk
Status: Open
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Iñaki Baz (ibc_sf)
Assigned to: Sergio Gutierrez (saguti)
Summary: "strncmp" should not be used to match parameters, SDP...
Initial Comment:
The following code lines use case sensitive comparisions while most of them try
to match SIP parameters or SDP attributes, that whould be case insenstive per
SIP syntax:
modules/msilo/msilo.c: if(!ctaddr.s || ctaddr.len < 6 ||
strncmp(ctaddr.s, "sip:", 4)
modules/pua_bla/notify.c: if(strncmp(sep+1, "expires=",
8)!= 0)
modules/mediaproxy/mediaproxy.c: if (strncmp(uri.s, "sip:", 4)==0) {
modules/mediaproxy/mediaproxy.c: if (strncmp(uri.s, "sip:", 4)==0) {
modules/mediaproxy/mediaproxy.c: if (strncmp(line.s, "sendrecv",
8)==0 || strncmp(line.s, "sendonly", 8)==0 ||
modules/mediaproxy/mediaproxy.c: strncmp(line.s, "recvonly",
8)==0 || strncmp(line.s, "inactive", 8)==0) {
modules/rls/subscribe.c: if(ev_param->name.len== 2 &&
strncmp(ev_param->name.s, "id", 2)== 0)
modules/rls/subscribe.c: if(strncmp(hdr->body.s+ i,
"eventlist", 9)== 0)
modules/rls/resource_notify.c: if(strncmp(smc+1, "reason=", 7))
modules/rls/resource_notify.c: if(strncmp(smc+1, "expires=", 8))
modules/uri/checks.c: if (strncmp(ruri->s, "tel:", 4) != 0) return 1;
modules/pua/hash.h: if (strncmp(event->s, "presence", 8) == 0)
modules/pua/hash.h: if (strncmp(event->s, "xcap-diff", 9) == 0)
modules/pua/hash.h: if (strncmp(event->s, "dialog;sla", 10) == 0)
modules/pua/hash.h: if (strncmp(event->s, "conference", 10) == 0)
modules/pua/hash.h: if (strncmp(event->s, "presence;winfo", 14) == 0)
modules/pua/hash.h: if (strncmp(event->s, "message-summary", 15) ==
0)
modules/pua_xmpp/simple2xmpp.c:
(strncmp(msg->event->body.s,"presence",8 )==0))
modules/pua_xmpp/simple2xmpp.c:
(strncmp(msg->event->body.s,"presence.winfo",14 )==0))
modules/pua_xmpp/simple2xmpp.c: if(hdr &&
strncmp(hdr->body.s,"terminated", 10)== 0)
modules/pua_xmpp/simple2xmpp.c:
if(strncmp(hdr->body.s+11,"reason=timeout", 14)== 0)
modules/pua_xmpp/simple2xmpp.c: if(hdr &&
strncmp(hdr->body.s,"terminated", 10)== 0)
modules/uac/auth_hdr.c: if(val.len>=4 &&
!strncmp(val.s, "auth", 4))
modules/xmpp/xode_str.c: if (strncmp(&buf[i],"&",5)==0)
modules/xmpp/xode_str.c: } else if (strncmp(&buf[i],""",6)==0) {
modules/xmpp/xode_str.c: } else if (strncmp(&buf[i],"'",6)==0) {
modules/xmpp/xode_str.c: } else if (strncmp(&buf[i],"<",4)==0) {
modules/xmpp/xode_str.c: } else if (strncmp(&buf[i],">",4)==0) {
modules/presence_mwi/add_events.c: if (strncmp(body.s, "Messages-Waiting",
16) != 0) goto err;
modules/presence_mwi/add_events.c: if (strncmp(at, "yes", 3) == 0) at = at +
3;
modules/presence_mwi/add_events.c: if (strncmp(at, "no", 2) == 0) at = at
+ 2;
modules/imc/imc_cmd.c: if(cmd->param[0].len<4 || strncmp(cmd->param[0].s,
"sip:", 4)!=0)
modules/imc/imc_cmd.c: if(cmd->param[0].len<=4 || strncmp(cmd->param[0].s,
"sip:", 4)!=0)
modules/speeddial/sdlookup.c: if(user_s.len<4 || strncmp(user_s.s, "sip:", 4))
modules/jabber/xode_str.c: if (strncmp(&buf[i],"&",5)==0)
modules/jabber/xode_str.c: } else if
(strncmp(&buf[i],""",6)==0) {
modules/jabber/xode_str.c: } else if
(strncmp(&buf[i],"'",6)==0) {
modules/jabber/xode_str.c: } else if (strncmp(&buf[i],"<",4)==0) {
modules/jabber/xode_str.c: } else if (strncmp(&buf[i],">",4)==0) {
modules/nathelper/nathelper.c: if (strncmp(pnode->rn_address, "udp:",
4) == 0) {
modules/presence/subscribe.c: if(ev_param->name.len== 2 &&
strncmp(ev_param->name.s, "id", 2)== 0)
modules/presence/utils_func.h: if(strncmp(ip.s, "sip:", 4)!=0)
modules/presence/event_list.c: if(sep && strncmp(sep+1, "winfo", 5)== 0)
parser/parse_event.c: _e->params->name.len== 3 &&
strncmp(_e->params->name.s, "sla", 3)== 0 )
parser/sdp/sdp_helpr_funcs.c: if (strncmp(body->s, "a=rtpmap:", 9) !=0) {
parser/sdp/sdp_helpr_funcs.c: if ( !( (strncmp(cp1, "a=sendrecv", 10) == 0) ||
parser/sdp/sdp_helpr_funcs.c: (strncmp(cp1, "a=inactive", 10) == 0) ||
parser/sdp/sdp_helpr_funcs.c: (strncmp(cp1, "a=recvonly", 10) == 0) ||
parser/sdp/sdp_helpr_funcs.c: (strncmp(cp1, "a=sendonly", 10) == 0)
)) {
----------------------------------------------------------------------
>Comment By: Bogdan-Andrei Iancu (bogdan_iancu)
Date: 2008-11-18 11:11
Message:
Hi Sergio,
The patch looks good - please apply it on trunk and 1.4 (note you have an
invalid diff file in your tar - entries.diff - remove that as it is for a
SVN file).
Thanks and regards,
Bogdan
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=2204065&group_id=232389
_______________________________________________
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
