kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-21 20:40:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove unneeded if(params) checks in reqresp_parser.

Browse Source 
Nick Lewis added them in https://reviewboard.asterisk.org/r/549/diff/1-2/
for no apparent reason. There is no way that params could become NULL in
that piece of code, so I removed these excess checks again.
........

Merged revisions 344837 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 344839 from http://svn.asterisk.org/svn/asterisk/branches/10


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@344840 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Walter Doekes
2011-11-11 21:37:53 +00:00
parent bac9ff62ef
commit 863d7189b9
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
channels/sip/reqresp_parser.c
View File

@@ -176,23 +176,23 @@ int parse_uri_full(char *uri, const char *scheme, char **user, char **pass,
}
if (!strcmp(label, "transport")) {
if (params) {params->transport=value;}
params->transport = value;
rem = parameters;
} else if (!strcmp(label, "user")) {
if (params) {params->user=value;}
params->user = value;
rem = parameters;
} else if (!strcmp(label, "method")) {
if (params) {params->method=value;}
params->method = value;
rem = parameters;
} else if (!strcmp(label, "ttl")) {
if (params) {params->ttl=value;}
params->ttl = value;
rem = parameters;
} else if (!strcmp(label, "maddr")) {
if (params) {params->maddr=value;}
params->maddr = value;
rem = parameters;
/* Treat "lr", "lr=yes", "lr=on", "lr=1", "lr=almostanything" as lr enabled and "", "lr=no", "lr=off", "lr=0", "lr=" and "lranything" as lr disabled */
} else if ((!strcmp(label, "lr") && strcmp(value, "no") && strcmp(value, "off") && strcmp(value, "0") && strcmp(value, "")) || ((lr) && strcmp(value, "lr"))) {
if (params) {params->lr=1;}
params->lr = 1;
rem = parameters;
} else {
value--;