Change in osmo-mgw[master]: mgcp_internal: LOGPENDP: ensure *endp is not NULL

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

lynxis lazus gerrit-no-reply at lists.osmocom.org
Thu May 2 16:00:20 UTC 2019


lynxis lazus has uploaded this change for review. ( https://gerrit.osmocom.org/13849


Change subject: mgcp_internal: LOGPENDP: ensure *endp is not NULL
......................................................................

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
---
M include/osmocom/mgcp/mgcp_internal.h
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.osmocom.org:29418/osmo-mgw refs/changes/49/13849/1

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...) \

-- 
To view, visit https://gerrit.osmocom.org/13849
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie9b5ecf08f69533ccb2fbd7fbbb529105e0c922f
Gerrit-Change-Number: 13849
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190502/7c67e818/attachment.htm>


More information about the gerrit-log mailing list