diff options
| author | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2021-06-06 15:47:49 +0200 |
|---|---|---|
| committer | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2021-06-06 15:50:42 +0200 |
| commit | 435f67f43c4323702b6c59d74017860e6337dd53 (patch) | |
| tree | 214d992aed35483b2c197f870fa07794170f0586 | |
| parent | 2e21a68e837f7a9098b4aeb1b806f51764bffc6f (diff) | |
msc_tx_common_id(): fix potential NULL pointer dereference
Reported by GCC 11.1.0. msc_a_vsub() may return NULL.
Change-Id: Iebdd6399e819a03258398e6b7b453bda37e45a20
| -rw-r--r-- | src/libmsc/msc_a.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/libmsc/msc_a.c b/src/libmsc/msc_a.c index 391d2510b..05030d307 100644 --- a/src/libmsc/msc_a.c +++ b/src/libmsc/msc_a.c @@ -1638,6 +1638,8 @@ struct msc_a *msc_a_for_vsub(const struct vlr_subscr *vsub, bool valid_conn_only int msc_tx_common_id(struct msc_a *msc_a, enum msc_role to_role) { struct vlr_subscr *vsub = msc_a_vsub(msc_a); + if (vsub == NULL) + return -ENODEV; struct ran_msg msg = { .msg_type = RAN_MSG_COMMON_ID, .common_id = { |