[PATCH] ipa: Be consistent in checking conn->cfg (Coverity)

[Jacob Erlbeck]

Coverity complains about checking connection->cfg in
bsc_close_connection() at one place but not at the second.

This patch fixes this by adding a check before accessing cfg when
generating the 'partial message' log message.

Fixes: Coverity CID 1195180
Sponsored-by: On-Waves ehf
---
 openbsc/src/osmo-bsc_nat/bsc_nat.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openbsc/src/osmo-bsc_nat/bsc_nat.c 
b/openbsc/src/osmo-bsc_nat/bsc_nat.c
index 524186a..9defbd2 100644
--- a/openbsc/src/osmo-bsc_nat/bsc_nat.c
+++ b/openbsc/src/osmo-bsc_nat/bsc_nat.c
@@ -916,7 +916,7 @@ void bsc_close_connection(struct bsc_connection *connection)
 
        if (connection->pending_msg) {
                LOGP(DNAT, LOGL_ERROR, "Dropping partial message on connection 
%d.\n",
-                    connection->cfg->nr);
+                    connection->cfg ? connection->cfg->nr : -1);
                msgb_free(connection->pending_msg);
                connection->pending_msg = NULL;
        }
-- 
1.7.9.5