Opened 3 months ago

Closed 3 months ago

#23054 closed defect (fixed)

BUG() macros shouldn't be warned as dead-code under coverity.

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

Description

It's fine for our BUG() checks to be dead code: they're functionally equivalent to nonfatal assertions. Leaving them in is good defensive programming, since they prevent us from introducing bugs later on.

Child Tickets

Change History (1)

comment:1 Changed 3 months ago by nickm

Resolution: fixed
Status: newclosed

Fixed in 602c52cad486516defc9d5ed375effd74cfe4529

Note: See TracTickets for help on using tickets.