Opened 5 days ago

Last modified 4 days ago

#30189 needs_review defect

ALL_BUGS_ARE_FATAL build failures in 0.3.4 and later.

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.0.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: 035-backport
Cc: Actual Points: .1
Parent ID: Points:
Reviewer: Sponsor:

Description

Found while reviewing #30179:

We don't include stdlib.h in util_bugs.h, which is sometimes a problem, since the macros declared there can call abort().

This problem is particularly bad for the nonfatal assertions, since they don't call abort() unless ALL_BUGS_ARE_FATAL is defined, which it only rarely is.

Child Tickets

Change History (5)

comment:1 Changed 5 days ago by nickm

Actual Points: .1
Component: - Select a componentCore Tor/Tor
Status: assignedneeds_review
Last edited 5 days ago by nickm (previous) (diff)

comment:2 Changed 5 days ago by nickm

There's a merge conflict with master, resolved in bug30189_041 with PR at https://github.com/torproject/tor/pull/953

I recommend no backport to 0.3.4, since this is a pretty rare configuration for anybody to build in.

comment:3 Changed 5 days ago by nickm

(Note that master will still be broken with ALL_BUGS_ARE_FATAL until #30179 is merged on top of this)

comment:4 Changed 5 days ago by teor

Should we add ALL_BUGS_ARE_FATAL to one of our CI jobs?

comment:5 Changed 4 days ago by nickm

On the one hand it might be worthwhile, but on the other hand it's not actually a configuration we recommend for anything but stress-testing, fuzzing, and the like.

Note: See TracTickets for help on using tickets.