fix for coverity 182
this bug can not currently trigger but if someone would rename the module in the future then this could potentially cause a null dereference. svn path=/trunk/; revision=18494
This commit is contained in:
parent
847090fe54
commit
85f49f1d49
10
epan/prefs.c
10
epan/prefs.c
|
@ -2005,10 +2005,12 @@ set_pref(gchar *pref_name, gchar *value)
|
||||||
} else if (strcmp(module->name, "smpp") == 0) {
|
} else if (strcmp(module->name, "smpp") == 0) {
|
||||||
/* Handle preferences that moved from SMPP. */
|
/* Handle preferences that moved from SMPP. */
|
||||||
module_t *new_module = find_module("gsm-sms-ud");
|
module_t *new_module = find_module("gsm-sms-ud");
|
||||||
if (strcmp(dotp, "port_number_udh_means_wsp") == 0)
|
if(new_module){
|
||||||
pref = find_preference(new_module, "port_number_udh_means_wsp");
|
if (strcmp(dotp, "port_number_udh_means_wsp") == 0)
|
||||||
else if (strcmp(dotp, "try_dissect_1st_fragment") == 0)
|
pref = find_preference(new_module, "port_number_udh_means_wsp");
|
||||||
pref = find_preference(new_module, "try_dissect_1st_fragment");
|
else if (strcmp(dotp, "try_dissect_1st_fragment") == 0)
|
||||||
|
pref = find_preference(new_module, "try_dissect_1st_fragment");
|
||||||
|
}
|
||||||
} else if (strcmp(module->name, "asn1") == 0) {
|
} else if (strcmp(module->name, "asn1") == 0) {
|
||||||
/* Handle old generic ASN.1 preferences (it's not really a
|
/* Handle old generic ASN.1 preferences (it's not really a
|
||||||
rename, as the new preferences support multiple ports,
|
rename, as the new preferences support multiple ports,
|
||||||
|
|
Loading…
Reference in New Issue