Project

General

Profile

Actions

Bug #2330

closed

add --enable-sanitize configure flag to osmocom cellular network projects (osmo-{msc,bsc,sgsn} and dependencies)

Added by neels almost 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Urgent
Assignee:
Target version:
-
Start date:
06/18/2017
Due date:
% Done:

100%

Spec Reference:

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

Related to OsmoHNBGW - Bug #2638: segfault during IuCS connect requestClosedneels11/14/2017

Actions
Actions #2

Updated by msuraev almost 7 years ago

In libosmo-abis it was added in 8a3be282ab265a2587608e471b274dda200eae84.

Actions #3

Updated by laforge over 6 years ago

  • Assignee set to neels
  • Priority changed from Normal to Low
Actions #4

Updated by neels over 6 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.

Actions #5

Updated by neels over 6 years ago

  • Description updated (diff)
Actions #6

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.

Actions #7

Updated by neels over 6 years ago

  • Related to Bug #2638: segfault during IuCS connect request added
Actions #8

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.

Actions #9

Updated by laforge about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)