sms: Possiqble meamleak fix gsm340_rx_tpdu()
Slight clean up of the code in gsm340_rx_tpdu() and a fix for an unlikely, but possible memory leak there.
This commit is contained in:
parent
a3d41c9124
commit
4ad593c8f6
|
@ -337,8 +337,8 @@ static int gsm340_rx_tpdu(struct gsm_subscriber_connection *conn, struct msgb *m
|
||||||
|
|
||||||
sms_alphabet = gsm338_get_sms_alphabet(gsms->data_coding_scheme);
|
sms_alphabet = gsm338_get_sms_alphabet(gsms->data_coding_scheme);
|
||||||
if (sms_alphabet == 0xffffffff) {
|
if (sms_alphabet == 0xffffffff) {
|
||||||
sms_free(gsms);
|
rc = GSM411_RP_CAUSE_MO_NET_OUT_OF_ORDER;
|
||||||
return GSM411_RP_CAUSE_MO_NET_OUT_OF_ORDER;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
switch (sms_vpf) {
|
switch (sms_vpf) {
|
||||||
|
@ -359,7 +359,8 @@ static int gsm340_rx_tpdu(struct gsm_subscriber_connection *conn, struct msgb *m
|
||||||
default:
|
default:
|
||||||
LOGP(DLSMS, LOGL_NOTICE,
|
LOGP(DLSMS, LOGL_NOTICE,
|
||||||
"SMS Validity period not implemented: 0x%02x\n", sms_vpf);
|
"SMS Validity period not implemented: 0x%02x\n", sms_vpf);
|
||||||
return GSM411_RP_CAUSE_MO_NET_OUT_OF_ORDER;
|
rc = GSM411_RP_CAUSE_MO_NET_OUT_OF_ORDER;
|
||||||
|
goto out;
|
||||||
}
|
}
|
||||||
gsms->user_data_len = *smsp++;
|
gsms->user_data_len = *smsp++;
|
||||||
if (gsms->user_data_len) {
|
if (gsms->user_data_len) {
|
||||||
|
|
Loading…
Reference in New Issue