fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30099 ) Change subject: trxcon: use L1CTL_HEADROOM in l1ctl_alloc_msg() ...................................................................... trxcon: use L1CTL_HEADROOM in l1ctl_alloc_msg() The l1ctl_client_read_cb() is using L1CTL_HEADROOM, so let's avoid inconsistency and use L1CTL_HEADROOM in the l1ctl_alloc_msg() too. Change-Id: I248f8c662ae0836cba61fb708a5dc73c57131f4c Related: OS#5599 --- M src/host/trxcon/src/l1ctl.c 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/99/30099/1 diff --git a/src/host/trxcon/src/l1ctl.c b/src/host/trxcon/src/l1ctl.c index 4923899..0075fcc 100644 --- a/src/host/trxcon/src/l1ctl.c +++ b/src/host/trxcon/src/l1ctl.c @@ -62,8 +62,8 @@ * Each L1CTL message gets its own length pushed in front * before sending. This is why we need this small headroom. */ - msg = msgb_alloc_headroom(L1CTL_LENGTH + L1CTL_MSG_LEN_FIELD, - L1CTL_MSG_LEN_FIELD, "l1ctl_tx_msg"); + msg = msgb_alloc_headroom(L1CTL_LENGTH + L1CTL_HEADROOM, + L1CTL_HEADROOM, "l1ctl_tx_msg"); if (!msg) { LOGP(DL1C, LOGL_ERROR, "Failed to allocate memory\n"); return NULL; -- To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/30099 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Change-Id: I248f8c662ae0836cba61fb708a5dc73c57131f4c Gerrit-Change-Number: 30099 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-MessageType: newchange