pespin submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
select.c: Check for ENABLE_PSEUDOTALLOC instead of EMBEDDED

The reason to avoid that code is that pseudotalloc doesn't implement
that function. Hence check for pseudotalloc support specifically.

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

diff --git a/src/core/select.c b/src/core/select.c
index 267f041..a577c5b 100644
--- a/src/core/select.c
+++ b/src/core/select.c
@@ -509,12 +509,12 @@
int osmo_select_main(int polling)
{
int rc = _osmo_select_main(polling);
-#ifndef EMBEDDED
+#ifndef ENABLE_PSEUDOTALLOC
if (talloc_total_size(osmo_ctx->select) != 0) {
osmo_panic("You cannot use the 'select' volatile "
"context if you don't use osmo_select_main_ctx()!\n");
}
-#endif
+#endif /* ifndef ENABLE_PSEUDOTALLOC */
return rc;
}


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

Gerrit-MessageType: merged
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I9c4aa66a55beac710912db9b8d3ad9ae5c1679ef
Gerrit-Change-Number: 41869
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>