commit 9d9f44ac718498c1cac8cb44a649f177129f7546
author Alexander Couzens <lynxis@fe80.eu> Thu May 02 17:48:13 2019 +0200
committer Alexander Couzens <lynxis@fe80.eu> Thu May 02 17:59:51 2019 +0200
tree df1eb53fe48ef3b9ea4a7dd4d5e42ee8a891c983
parent 84274e4e9baf358409dd8df214e1e075ee6f7458

mgcp_internal: LOGPENDP: ensure *endp is not NULL

In certain cases the endp can be NULL. LOGPENDP will dereference the
pointer to retreive the endpoint number.

Fixes: 8a893442a1e9 ("mgcp_internal: LOGPENDP: ensure *endp is not NULL")
Change-Id: Ie9b5ecf08f69533ccb2fbd7fbbb529105e0c922f

include/osmocom/mgcp/mgcp_internal.h

diff --git a/include/osmocom/mgcp/mgcp_internal.h b/include/osmocom/mgcp/mgcp_internal.h
index 24e28b4..82ff6c8 100644
--- a/include/osmocom/mgcp/mgcp_internal.h
+++ b/include/osmocom/mgcp/mgcp_internal.h
@@ -338,7 +338,7 @@
 
 #define LOGPENDP(endp, cat, level, fmt, args...) \
 LOGP(cat, level, "endpoint:0x%x " fmt, \
-     ENDPOINT_NUMBER(endp), \
+     endp ? ENDPOINT_NUMBER(endp) : -1, \
      ## args)
 
 #define LOGPCONN(conn, cat, level, fmt, args...) \