Page MenuHomePhabricator

focus_manager: fix memory leaks
ClosedPublic

Authored by ManMower on Jan 14 2019, 5:06 PM.

Details

Summary

_set_a_without_b() makes a list clone internally, so cloning the list
first will leak a copy.
Depends on D7621

Diff Detail

Repository
rEFL core/efl
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ManMower created this revision.Jan 14 2019, 5:06 PM

It seems that this patch has no reviewers specified. If you are unsure who can review your patch, please check this wiki page and see if anyone can be added: https://phab.enlightenment.org/w/maintainers_reviewers/

ManMower requested review of this revision.Jan 14 2019, 5:06 PM

Adding bu5hm4n on review as he knows this code better than anyone... Slap me if I've missed something obvious here!

bu5hm4n accepted this revision.Jan 14 2019, 11:45 PM

Looks good to me :)

This revision is now accepted and ready to land.Jan 14 2019, 11:45 PM
This revision was automatically updated to reflect the committed changes.