Bug #2330
closedadd --enable-sanitize configure flag to osmocom cellular network projects (osmo-{msc,bsc,sgsn} and dependencies)
100%
Description
Let's add the --enable-sanitize configure switch in every osmocom git repos (of interest) and include such build in the standard master HEAD as well as gerrit build jobs. We could then drop the Osmocom_Sanitize build entirely.
Related issues
Updated by neels almost 7 years ago
ml thread: http://lists.osmocom.org/pipermail/openbsc/2017-June/010821.html
add --enable-sanitize in libosmocore: http://git.osmocom.org/libosmocore/commit/?id=a23817622b28cb1969a73ffd36da501eb29b9cd7
Updated by msuraev almost 7 years ago
In libosmo-abis it was added in 8a3be282ab265a2587608e471b274dda200eae84.
Updated by laforge almost 7 years ago
- Assignee set to neels
- Priority changed from Normal to Low
Updated by neels almost 7 years ago
- Subject changed from fix Osmocom_Sanitize build to add --enable-sanitize configure flag to osmocom cellular network projects (osmo-{msc,bsc,sgsn} and dependencies)
Actually, the fact that 'make CFLAGS+=...' breaks the build was due to an error in Makefile.am, which was using CFLAGS instead of AM_CFLAGS: fixed in https://gerrit.osmocom.org/2975
It would still be nice to have --enable-sanitize everywhere.
Updated by neels over 6 years ago
- Status changed from New to In Progress
- Priority changed from Low to Urgent
I am facing segfaults in osmo-hnbgw (#2638) and started to add sanitizer options to osmo-iuh.git and osmo-msc.git. Actually, the regression tests uncover sanitizer failures right away. So adding sanitizer options now and fixing the bugs.
Updated by neels over 6 years ago
- Related to Bug #2638: segfault during IuCS connect request added
Updated by neels over 6 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
I've added an --enable-sanitize configure flag to all cellNet git trees (merged).
https://gerrit.osmocom.org/#/q/%22--enable-sanitize%22
Found scores of new sanitizer issues using debian9 (stable), which finds more issues than deb8 did.