laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve pespin: Looks good to me, approved Jenkins Builder: Verified
msgb: do not use msgb_l4 instead of msgb_sms

The macro msgb_sms() is basically an alias for msgb_l4(). Lets use
msgb_l4() in msgb_l4len() instead of msgb_sms().

Change-Id: I90d4f5c07fbaadd9e022752a2c64c4855f0b4227
---
M include/osmocom/core/msgb.h
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/osmocom/core/msgb.h b/include/osmocom/core/msgb.h
index fbf1742..1e2b023 100644
--- a/include/osmocom/core/msgb.h
+++ b/include/osmocom/core/msgb.h
@@ -184,7 +184,7 @@
static inline unsigned int msgb_l4len(const struct msgb *msgb)
{
OSMO_ASSERT(msgb->l4h);
- return msgb->tail - (uint8_t *)msgb_sms(msgb);
+ return msgb->tail - (uint8_t *)msgb_l4(msgb);
}

/*! determine the length of the header

To view, visit change 29620. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I90d4f5c07fbaadd9e022752a2c64c4855f0b4227
Gerrit-Change-Number: 29620
Gerrit-PatchSet: 2
Gerrit-Owner: dexter <pmaier@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged