pespin submitted this change.

View Change


Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved fixeria: Looks good to me, approved
select.c: Clarify osmo_fd_unregister() can only be called on registered osmo_fds

Change-Id: I5e397e121f2fe2254c7f4b474e6eefd7aebe7a83
---
M src/core/select.c
1 file changed, 10 insertions(+), 0 deletions(-)

diff --git a/src/core/select.c b/src/core/select.c
index 584de24..67b0fd3 100644
--- a/src/core/select.c
+++ b/src/core/select.c
@@ -212,6 +212,7 @@
/*! Unregister a file descriptor from select loop abstraction
* \param[in] fd osmocom file descriptor to be unregistered
*
+ * Caller is responsible for ensuring the fd is really registered before calling this API.
* This function must be called before changing the value of the fd field in
* the struct osmo_fd.
*/

To view, visit change 31902. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I5e397e121f2fe2254c7f4b474e6eefd7aebe7a83
Gerrit-Change-Number: 31902
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged