Opened 7 years ago

Closed 5 years ago

#9904 closed bug (fixed)

[Screen Savers/spider] SpiderSaver.cpp: Called C++ object pointer is null

Reported by: mt Owned by: korli
Priority: normal Milestone: R1
Component: Add-Ons/Screen Savers Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: yes Platform: All

Description

In SpiderSaver::_DrawInto(), if fQueues[i] is NULL (fails to allocate in _Init()), calling fQueues[i]->Head() and fQueues[i]->Tail() is invalid. Please see attached report.

Attachments (2)

0019-Fix-called-C-object-pointer-is-null.patch (835 bytes ) - added by mt 7 years ago.
Patch
report-d3b55a.html (60.6 KB ) - added by mt 7 years ago.
Report by Clang Static Analyzer

Download all attachments as: .zip

Change History (5)

comment:1 by mt, 7 years ago

Has a Patch: set

by mt, 7 years ago

Attachment: report-d3b55a.html added

Report by Clang Static Analyzer

comment:2 by korli, 7 years ago

There are a few problems with allocations to solve anyway. What can exactly be done when a failure happen on a parameter change? We have already deallocated things for the previous parameter value (no rollback is possible).

comment:3 by pulkomandy, 5 years ago

Resolution: fixed
Status: newclosed

Applied in hrev48344.

Note: See TracTickets for help on using tickets.