[PATCH] osmo-pcu[master]: Remove ASAN preprocessor warning

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/.

Max gerrit-no-reply at lists.osmocom.org
Tue Jan 23 20:03:01 UTC 2018


Review at  https://gerrit.osmocom.org/6001

Remove ASAN preprocessor warning

We have ASAN integrated into CI tests directly so there's no need in
additional manual warnings at compile time.

Change-Id: Idaabb4203a282be6779d7ab29de3f827b0ca6462
---
M src/gprs_bssgp_pcu.cpp
1 file changed, 2 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-pcu refs/changes/01/6001/1

diff --git a/src/gprs_bssgp_pcu.cpp b/src/gprs_bssgp_pcu.cpp
index a86e09d..a9d4617 100644
--- a/src/gprs_bssgp_pcu.cpp
+++ b/src/gprs_bssgp_pcu.cpp
@@ -964,10 +964,9 @@
 
 	/* FIXME: move this to libgb: btsctx_free() */
 	llist_del(&the_pcu.bctx->list);
-#warning "This causes ASAN to complain. It is not critical for normal operation but should be fixed nevertheless"
-#if 0
+
 	talloc_free(the_pcu.bctx);
-#endif
+
 	the_pcu.bctx = NULL;
 }
 

-- 
To view, visit https://gerrit.osmocom.org/6001
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idaabb4203a282be6779d7ab29de3f827b0ca6462
Gerrit-PatchSet: 1
Gerrit-Project: osmo-pcu
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>



More information about the gerrit-log mailing list