This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
pespin gerrit-no-reply at lists.osmocom.orgpespin has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-abis/+/19257 ) Change subject: ipaccess: Drop e1inp_line reference in ipacess_drop() ...................................................................... ipaccess: Drop e1inp_line reference in ipacess_drop() the bfd->data handles a reference to the e1np_line (obtained through e1inp_line_get() during bfd setup), so remember to drop it under this condition. Related: OS#4624 Change-Id: I418064df04872befe2e936e21768b6ea01263120 --- M src/input/ipaccess.c 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-abis refs/changes/57/19257/1 diff --git a/src/input/ipaccess.c b/src/input/ipaccess.c index 5d17839..08b34a4 100644 --- a/src/input/ipaccess.c +++ b/src/input/ipaccess.c @@ -92,6 +92,13 @@ osmo_fd_unregister(bfd); close(bfd->fd); bfd->fd = -1; + /* This is BSC code, ipaccess_drop() is only called for + accepted() sockets, hence the bfd holds a reference to + e1inp_line in ->data that needs to be released */ + OSMO_ASSERT(bfd->data == line); + bfd->data = NULL; + e1inp_line_put2(line, "ipa_bfd"); + ret = -ENOENT; } -- To view, visit https://gerrit.osmocom.org/c/libosmo-abis/+/19257 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-abis Gerrit-Branch: master Gerrit-Change-Id: I418064df04872befe2e936e21768b6ea01263120 Gerrit-Change-Number: 19257 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin at sysmocom.de> Gerrit-MessageType: newchange -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20200714/fb6bc7cd/attachment.htm>