New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

libaudqt: Fix crash when infopopup is being hidden #32

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@awilfox
Copy link

awilfox commented Sep 20, 2018

Hovering over the track I wanted to listen to showed a tool tip (the
InfoPopup widget); if I moved the mouse in any way after the widget was
displayed, it would immediately crash with a segmentation fault.

Valgrind showed that this was due to s_infopopup being deleted from an
event that involved it. By switching both delete to deleteLater,
the crash no longer occurs.

Closes: #828.

libaudqt: Fix crash when infopopup is being hidden
Hovering over the track I wanted to listen to showed a tool tip (the
InfoPopup widget); if I moved the mouse in any way after the widget was
displayed, it would immediately crash with a segmentation fault.

Valgrind showed that this was due to s_infopopup being deleted from an
event that involved it.  By switching both `delete` to `deleteLater`,
the crash no longer occurs.

Closes: #828.
@jlindgren90

This comment has been minimized.

Copy link
Member

jlindgren90 commented Sep 20, 2018

Adjusted slightly and applied. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment