summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHolger Hans Peter Freyther <holger@moiji-mobile.com>2013-09-04 08:45:48 +0200
committerHolger Hans Peter Freyther <holger@moiji-mobile.com>2013-09-04 08:45:48 +0200
commit3ffd9bc0a4efe03022d87513dbde3efc7b4fb8e6 (patch)
treefe90cb71d85ed7a528109d91b7e252d4bf288450
parent1b9902c1280b3b1092b726695142a09a1edcb154 (diff)
bsc_msc: Fix possible null pointer dereference
In case the allocation of con is failing, do not attempt to print con->name as this will be a null pointer dereference. Fixes: Coverity CID 1076318
-rw-r--r--openbsc/src/libbsc/bsc_msc.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/openbsc/src/libbsc/bsc_msc.c b/openbsc/src/libbsc/bsc_msc.c
index c8cdce0..1a0f78a 100644
--- a/openbsc/src/libbsc/bsc_msc.c
+++ b/openbsc/src/libbsc/bsc_msc.c
@@ -226,9 +226,7 @@ struct bsc_msc_connection *bsc_msc_create(void *ctx, struct llist_head *dests)
con = talloc_zero(NULL, struct bsc_msc_connection);
if (!con) {
- LOGP(DMSC, LOGL_FATAL,
- "Failed to create the MSC(%s) connection.\n",
- con->name);
+ LOGP(DMSC, LOGL_FATAL, "Failed to create the MSC connection.\n");
return NULL;
}