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 <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <[email protected]>
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: lynxis lazus <[email protected]>

Reply via email to