osmith has submitted this change. (
https://gerrit.osmocom.org/c/osmo-iuh/+/31587 )
Change subject: test-ranap: don't use libosmocore's tall_msgb_ctx
......................................................................
test-ranap: don't use libosmocore's tall_msgb_ctx
Use msgb_ctx from test_common.c instead, which is the same pointer after
test_common_init() ran. tall_msgb_ctx from libosmocore is no longer
getting exported since I13169c00a59fb59513dfc598de5a71d094492422.
Fix for:
/usr/bin/ld: test-ranap.o: in function `main':
/build/src/tests/test-ranap.c:201: undefined reference to `tall_msgb_ctx'
/usr/bin/ld: /build/src/tests/test-ranap.c:201: undefined reference to
`tall_msgb_ctx'
Change-Id: I7d0b3068760c2cd7067e2a5d929f03c7e7d1decb
---
M src/tests/test-ranap.c
M src/tests/test_common.c
2 files changed, 21 insertions(+), 3 deletions(-)
Approvals:
daniel: Looks good to me, but someone else must approve
pespin: Looks good to me, but someone else must approve
osmith: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/tests/test-ranap.c b/src/tests/test-ranap.c
index 633bb2b..e593bd9 100644
--- a/src/tests/test-ranap.c
+++ b/src/tests/test-ranap.c
@@ -32,7 +32,7 @@
#include "test_common.h"
-extern void *tall_msgb_ctx;
+extern void *msgb_ctx;
static void test_aper_int(uint32_t inp)
{
@@ -198,7 +198,7 @@
printf("report\n");
talloc_report(talloc_asn1_ctx, stdout);
- talloc_report(tall_msgb_ctx, stdout);
+ talloc_report(msgb_ctx, stdout);
//talloc_report(NULL, stdout);
test_common_cleanup();
diff --git a/src/tests/test_common.c b/src/tests/test_common.c
index 187a269..36d9954 100644
--- a/src/tests/test_common.c
+++ b/src/tests/test_common.c
@@ -71,7 +71,7 @@
};
static void *tall_hnb_ctx;
-static void *msgb_ctx;
+void *msgb_ctx;
int test_common_init(void)
{
--
To view, visit
https://gerrit.osmocom.org/c/osmo-iuh/+/31587
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I7d0b3068760c2cd7067e2a5d929f03c7e7d1decb
Gerrit-Change-Number: 31587
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann(a)sysmocom.de>
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged