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

[dexter]

Hello Max, Neels Hofmeyr, Jenkins Builder,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/2011

to look at the new patch set (#2).

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(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/11/2011/2

diff --git a/src/input/lapd.c b/src/input/lapd.c
index f5909b4..6d7f426 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: newpatchset
Gerrit-Change-Id: I9b961d5e36967cfd953551b24696849541257a1c
Gerrit-PatchSet: 2
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>
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>