Add a TODO about a possible use after free bug.
This commit is contained in:
@@ -238,6 +238,9 @@ AppGroupView::AddInfo(NotificationView* view)
|
|||||||
|
|
||||||
for (int32 i = 0; i < children; i++) {
|
for (int32 i = 0; i < children; i++) {
|
||||||
if (id == fInfo[i]->MessageID()) {
|
if (id == fInfo[i]->MessageID()) {
|
||||||
|
// TODO: because NotificationWindow also tracks these
|
||||||
|
// views, we may be heading towards a use-after-free with
|
||||||
|
// this code.
|
||||||
GetLayout()->RemoveView(fInfo[i]);
|
GetLayout()->RemoveView(fInfo[i]);
|
||||||
delete fInfo[i];
|
delete fInfo[i];
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user