Skip to content

Commit

Permalink
Fixing memory leak with pertag
Browse files Browse the repository at this point in the history
  • Loading branch information
jzbor committed Apr 27, 2021
1 parent fa84910 commit f39b15a
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/moonwm.c
Original file line number Diff line number Diff line change
Expand Up @@ -771,6 +771,8 @@ cleanup(void)
for (m = mons; m; m = m->next)
while (m->stack)
unmanage(m->stack, 0);
while (systray->icons)
removesystrayicon(systray->icons);

This comment has been minimized.

Copy link
@bakkeby

bakkeby Apr 28, 2021

Contributor

If you set showsystray to 0 in your config.h then systray here will be NULL and your WM will crash when exiting, perhaps move this block into the if (showsystray) { block further down?

This comment has been minimized.

Copy link
@jzbor

jzbor Apr 28, 2021

Author Owner

Oh yeah - you are absolutely right. I didn't even notice there was a systray section. Thanks for the hint!

XUngrabKey(dpy, AnyKey, AnyModifier, root);
while (mons)
cleanupmon(mons);
Expand Down

0 comments on commit f39b15a

Please sign in to comment.