pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve osmith: Looks good to me, approved
sccp: Drop unused local variable

Change-Id: I7f829f1c285d01faeaad317262ddd51f39e42886
---
M src/sccp.c
1 file changed, 9 insertions(+), 3 deletions(-)

diff --git a/src/sccp.c b/src/sccp.c
index ecab9ab..85bea6d 100644
--- a/src/sccp.c
+++ b/src/sccp.c
@@ -840,7 +840,6 @@
struct msgb *request;
struct sccp_connection_request *req;
uint8_t *data;
- uint8_t extra_size = 3 + 1;
int called_len;

if (l3_data) {
@@ -850,8 +849,6 @@
}
}

- if (l3_data)
- extra_size += 2 + l3_length;
request = msgb_alloc_headroom(SCCP_MSG_SIZE,
SCCP_MSG_HEADROOM, "sccp connection request");
request->l2h = &request->data[0];

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

Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Change-Id: I7f829f1c285d01faeaad317262ddd51f39e42886
Gerrit-Change-Number: 34617
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged