diff options
author | Pau Espin Pedrol <pespin@sysmocom.de> | 2020-07-14 21:11:56 +0200 |
---|---|---|
committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2020-07-15 15:39:33 +0200 |
commit | 8737ad4f75331e68b94b15fd3da270e7335ba594 (patch) | |
tree | e962aa075605e82c9f2d97b64288a015b3957d80 /include | |
parent | b8ea0ff521a3e01c22a9dd1948b9a853521f575e (diff) |
ipacces: Fix e1inp_line reference put in ipaccess_close
Drop the function e1inp_close_socket since it's only used by the caller
at hand, and it's only exported through "internal.h", so no app is using
it. Remove it because there's only a caller, and furthermore because
keeping it (and putting bfd->data==line) would introduce a layer
violation because the bfd->data==line is only used for ipaccess so far.
Triggering path:
handle_ts1_read ret=0 "Sign link vanished"
ipaccess_drop
line->ops->sign_link_down
(osmo-bsc) ipaccess_drop_oml
e1inp_sign_link_destroy
link->ts->line->driver->close
ipaccess_close
Related: OS#4624
Change-Id: If23cc722106a9f70c998e591369a4acafa52c519
Diffstat (limited to 'include')
-rw-r--r-- | include/internal.h | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/include/internal.h b/include/internal.h index c931d4f..8a5aa2e 100644 --- a/include/internal.h +++ b/include/internal.h @@ -20,13 +20,7 @@ const char *e1inp_ipa_get_bind_addr(void); struct msgb; struct msgb *ipa_msg_alloc(int headroom); -/* - * helper for internal drivers, not public - */ -void e1inp_close_socket(struct e1inp_ts *ts, - struct e1inp_sign_link *sign_link, - struct osmo_fd *bfd); - +int e1inp_int_snd_event(struct e1inp_ts *ts, struct e1inp_sign_link *link, int evt); #endif |