diff options
| author | Andreas Eversberg <jolly@eversberg.eu> | 2023-10-02 15:44:38 +0200 |
|---|---|---|
| committer | Andreas Eversberg <jolly@eversberg.eu> | 2023-10-04 12:55:03 +0200 |
| commit | 1cc4c0f487a1c7b412c4ad46b386847fcba0512d (patch) | |
| tree | 7fc04847d39bbea491ca6336036c336385503604 /src/host/layer23/src | |
| parent | d6b2a4d6851d5d1615190e3f810500668bd83b50 (diff) | |
Fix potential NULL pointer dereferences, discovered by Coverity Scan
See: CID 323362 + CID 323363
Change-Id: I47aa506014d8bddb8c8ce9b506c1c5c7b8056d30
Diffstat (limited to 'src/host/layer23/src')
| -rw-r--r-- | src/host/layer23/src/mobile/gsm48_mm.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/host/layer23/src/mobile/gsm48_mm.c b/src/host/layer23/src/mobile/gsm48_mm.c index cec798ba..810dfa7b 100644 --- a/src/host/layer23/src/mobile/gsm48_mm.c +++ b/src/host/layer23/src/mobile/gsm48_mm.c @@ -3190,6 +3190,8 @@ static int gsm48_mm_init_mm(struct osmocom_ms *ms, struct msgb *msg, (msg_type & GSM48_MMXX_MASK), mmh->ref, mmh->transaction_id, sapi); + if (!nmsg) + return -ENOMEM; nmmh = (struct gsm48_mmxx_hdr *)nmsg->data; nmmh->cause = cause; gsm48_mmxx_upmsg(ms, nmsg); @@ -3410,6 +3412,8 @@ static int gsm48_mm_init_mm_reject(struct osmocom_ms *ms, struct msgb *msg) (msg_type & GSM48_MMXX_MASK), mmh->ref, mmh->transaction_id, sapi); + if (!nmsg) + return -ENOMEM; nmmh = (struct gsm48_mmxx_hdr *)nmsg->data; nmmh->cause = 17; gsm48_mmxx_upmsg(ms, nmsg); |