laforge has submitted this change. ( https://gerrit.osmocom.org/c/dahdi-tools/+/27274 ) Change subject: dahdi_pcap: Fix LAPD pseudo header hardware address length ...................................................................... dahdi_pcap: Fix LAPD pseudo header hardware address length the 'halen' field is the length of the hardware address (8 bytes), and not the length of the payload. Change-Id: I72e009f40c8e9e295bc1e1f4a0951cd1236cdf43 --- M dahdi_pcap.c 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified manawyrm: Looks good to me, approved diff --git a/dahdi_pcap.c b/dahdi_pcap.c index f6fa7fe..225fe78 100644 --- a/dahdi_pcap.c +++ b/dahdi_pcap.c @@ -163,7 +163,7 @@ lapd->sll_pkttype = 3; lapd->sll_hatype = 0; - lapd->sll_halen = res; + lapd->sll_halen = htons(8); // lapd->sll_addr = ??? lapd->sll_protocol[0] = 0x00; lapd->sll_protocol[1] = 0x30; -- To view, visit https://gerrit.osmocom.org/c/dahdi-tools/+/27274 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: dahdi-tools Gerrit-Branch: master Gerrit-Change-Id: I72e009f40c8e9e295bc1e1f4a0951cd1236cdf43 Gerrit-Change-Number: 27274 Gerrit-PatchSet: 3 Gerrit-Owner: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: manawyrm <osmocom.account(a)tbspace.de> Gerrit-Reviewer: roox <mardnh(a)gmx.de> Gerrit-MessageType: merged