diff options
author | Brian Ashworth <bosrsf04@gmail.com> | 2018-12-17 00:54:00 -0500 |
---|---|---|
committer | emersion <contact@emersion.fr> | 2018-12-17 09:21:14 +0100 |
commit | eb527ac01ad7e3255b757d620bc678a834a3958a (patch) | |
tree | a6b7eeb10108ec5b23aebbf9e43bf72e74fdefcb /contrib | |
parent | 8d7ebc258a830dc893bef327c2e672a901ced4ed (diff) |
swaynag: remove double free of details button
If there are no arguments or invalid arguments given, swaynag will free
`swaynag.details.button_details` under the `cleanup` label in main. It
then called `swaynag_destroy`, which would attempt to free it again.
Since `swaynag.details.button_details` is either freed on line 106 of
main (when there is no detailed message) or added to `swaynag.buttons`
on line 103 of main, there is no reason to manually free it in
`swaynag_destroy`.
Although I cannot reproduce a double free on my system, for some reason,
it should have actually resulted in a double free in all code paths.
Diffstat (limited to 'contrib')
0 files changed, 0 insertions, 0 deletions