Opened 8 days ago

Last modified 7 days ago

#31147 needs_review enhancement

Check tor_vasprintf for error return values.

Reported by: paldium Owned by:
Priority: Low Milestone: Tor: 0.4.2.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Normal Keywords: 035-backport 040-backport 041-backport
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


In case of error, a negative value will be returned or NULL written into
first supplied argument.

This patch uses both cases to comply with style in the specific files.

A tor_vasprintf error in process_vprintf would lead to a NULL dereference
later on in buf_add, because the return value -1 casted to size_t would
pass an assertion check inside of buf_add.

On the other hand, common systems will fail on such an operation, so it
is not a huge difference to a simple assertion. Yet it is better to
properly fail instead of relying on such behaviour on all systems.

Child Tickets

Attachments (1)

0001-Check-tor_vasprintf-for-error-return-values.patch (2.5 KB) - added by paldium 8 days ago.

Download all attachments as: .zip

Change History (2)

comment:1 Changed 7 days ago by nickm

Keywords: 035-backport 040-backport 041-backport added
Milestone: Tor: 0.4.2.x-final
Status: newneeds_review
Note: See TracTickets for help on using tickets.