Attention is currently required from: pespin.
jolly has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmo-abis/+/34035 )
Change subject: dahdi: Add missing msg->l2h
......................................................................
Patch Set 1:
(2 comments)
File src/input/dahdi.c:
https://gerrit.osmocom.org/c/libosmo-abis/+/34035/comment/d5cce9c9_64d4a754
PS1, Line 319: return ret;
msgb leak here.
Fixed in another patch.
https://gerrit.osmocom.org/c/libosmo-abis/+/34035/comment/00284afd_64db3a2a
PS1, Line 322: LOGPITS(e1i_ts, DLMI, LOGL_ERROR, "%s read failed %d (%s)\n",
__func__, ret, strerror(errno));
msgb leak here.
Fixed in another patch.
--
To view, visit
https://gerrit.osmocom.org/c/libosmo-abis/+/34035
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmo-abis
Gerrit-Branch: master
Gerrit-Change-Id: Ieec5e2e6bacc4bce465c29b2148537e8b2218a00
Gerrit-Change-Number: 34035
Gerrit-PatchSet: 1
Gerrit-Owner: jolly <andreas(a)eversberg.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Tue, 01 Aug 2023 10:44:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment