#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 16 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 16 months ago by asn

Status: newneeds_review

comment:3 Changed 16 months ago by asn

Reviewer: mikeperry

comment:4 Changed 15 months ago by mikeperry

Status: needs_reviewmerge_ready

lgtm

comment:5 Changed 14 months ago by nickm

Resolution: fixed
Status: merge_readyclosed

Branch merged.

Note: See TracTickets for help on using tickets.