Opened 11 years ago

Closed 10 years ago

#9875 closed bug (fixed)

[Interface kit] GridLayout.cpp: Memory allocated by 'new[]' should be deallocated by 'delete[]'

Reported by: mt Owned by: axeld
Priority: normal Milestone: R1
Component: Kits/Interface Kit Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

In BGridLayout::_ResizeGrid(), variable "grid[i]" should be deallocated by delete[]. Please see attached report.

Attachments (2)

0006-GridLayout.cpp-use-delete-instead-of-delete.patch​ (789 bytes ) - added by mt 11 years ago.
Patch
report-mq2NP5.html​ (93.0 KB ) - added by mt 11 years ago.
Report by Clang Static Analyzer

Download all attachments as: .zip

Change History (5)

by mt, 11 years ago

Attachment: 0006-GridLayout.cpp-use-delete-instead-of-delete.patch​ added

Patch

comment:1 by mt, 11 years ago

patch: 0 → 1

by mt, 11 years ago

Attachment: report-mq2NP5.html​ added

Report by Clang Static Analyzer

comment:2 by tangobravo, 11 years ago

Are these issues already reported by Coverity? It seems like a lot of effort to make bug reports for each of these issues - perhaps we should try and find a more efficient solution for you to get these fixes into the repository. Perhaps doing them directly in a git branch which someone with commit access can just rebase and commit?

comment:3 by mmadia, 10 years ago

Resolution: → fixed
Status: new → closed

Applied in hrev46351. Thanks for the patch!

Note: See TracTickets for help on using tickets.