Patch Set 2: Code-Review-1 (2 comments)
https://gerrit.osmocom.org/#/c/2667/2/src/host/layer23/src/mobile/gsm322.c File src/host/layer23/src/mobile/gsm322.c: Line 5067: LOGP(DCS, LOGL_ERROR, "Failed to read BA list: " my comment about talloc failures applies here, too. I thought I didn't have to mention this in my previous reply. https://gerrit.osmocom.org/#/c/2667/2/src/host/layer23/src/mobile/vty_interface.c File src/host/layer23/src/mobile/vty_interface.c: Line 1274: vty_out(vty, "Couldn't rename MS: no memory%s", VTY_NEWLINE); same here -- To view, visit https://gerrit.osmocom.org/2667 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Icd6706117fdd7f1b3481b0e3817bbb3b31f12f60 Gerrit-PatchSet: 2 Gerrit-Project: osmocom-bb Gerrit-Branch: master Gerrit-Owner: Vadim Yanitskiy <[email protected]> Gerrit-Reviewer: Harald Welte <[email protected]> Gerrit-HasComments: Yes
