[MERGED] libosmo-abis[master]: lapd: log pointer addresses

[Harald Welte]

Harald Welte has submitted this change and it was merged.

Change subject: lapd: log pointer addresses
......................................................................


lapd: log pointer addresses

Add pointer address output to the sap alloc/free functions in
order to track the objects later in the debug output (we have three
of them)

Change-Id: I9b961d5e36967cfd953551b24696849541257a1c
---
M src/input/lapd.c
1 file changed, 7 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/input/lapd.c b/src/input/lapd.c
index 370697d..4b5077b 100644
--- a/src/input/lapd.c
+++ b/src/input/lapd.c
@@ -229,8 +229,9 @@
 	if (!sap)
 		return NULL;
 
-	LOGP(DLLAPD, LOGL_NOTICE, "LAPD Allocating SAP for SAPI=%u / TEI=%u\n",
-		sapi, teip->tei);
+	LOGP(DLLAPD, LOGL_NOTICE,
+	     "LAPD Allocating SAP for SAPI=%u / TEI=%u (dl=%p, sap=%p)\n",
+	     sapi, teip->tei, &sap->dl, sap);
 
 	sap->sapi = sapi;
 	sap->tei = teip;
@@ -266,6 +267,10 @@
 /* Free SAP instance, including the datalink */
 static void lapd_sap_free(struct lapd_sap *sap)
 {
+	LOGP(DLLAPD, LOGL_NOTICE,
+	     "LAPD Freeing SAP for SAPI=%u / TEI=%u (dl=%p, sap=%p)\n",
+	     sap->sapi, sap->tei->tei, &sap->dl, sap);
+
 	/* free datalink structures and timers */
 	lapd_dl_exit(&sap->dl);
 

-- 
To view, visit https://gerrit.osmocom.org/2011
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I9b961d5e36967cfd953551b24696849541257a1c
Gerrit-PatchSet: 3
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>