Opened 11 months ago

Closed 9 months ago

#31027 closed defect (fixed)

Coverity: circpadding: dead code in circpad_machine_remove_token

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

Description

In circpad_machine_remove_token(), we check for state==NULL on line 1087, and then again on line 1107. Since state is not assigned between these points, the second check is dead code.

This is CID 1447298.

Child Tickets

Change History (5)

comment:1 Changed 11 months ago by asn

Please check the first commit of ​https://github.com/torproject/tor/pull/1160 which fixes both #31027 and #31024.

comment:2 Changed 11 months ago by asn

Status: newneeds_review

comment:3 Changed 11 months ago by asn

Reviewer: mikeperry

comment:4 Changed 10 months ago by mikeperry

Status: needs_reviewmerge_ready

lgtm

comment:5 Changed 9 months ago by nickm

Resolution: fixed
Status: merge_readyclosed

Branch merged.

Note: See TracTickets for help on using tickets.