Skip to content
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.

Sign up for GitHub

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

Jump to bottom

SurgeXT/Linux crash when click on about #4495

Closed
ghost opened this issue May 6, 2021 · 2 comments · Fixed by #4496
Closed

SurgeXT/Linux crash when click on about #4495

ghost opened this issue May 6, 2021 · 2 comments · Fixed by #4496
Labels
Bug Report Item submitted using the Bug Report template Linux Issues which only occur on Linux
Milestone
Surge XT 1.0

Comments

@ghost
Copy link

ghost commented May 6, 2021

Linux 64bit
Vst3 and Standalone
After #4494 clicking on "About Surge" leads to segfault

gdb output of a Standalone Debug build at 4d2f58c:
aboutbox_crash.txt
@mkruselj mkruselj added Bug Report Item submitted using the Bug Report template Linux Issues which only occur on Linux labels May 6, 2021
@mkruselj mkruselj added this to the Surge XT 1.0 milestone May 6, 2021
@baconpaul
Copy link
Collaborator

very useful thanks! will fix today

@baconpaul
Copy link
Collaborator

oh and it crashes all platforms - my fix to not leak gradients last night deals poorly with nullptr sets

baconpaul added a commit to baconpaul/surge that referenced this issue May 6, 2021
@baconpaul
Fix Gradient Crash; Fix install-local targets; update README
1574b23 
1. can't remember a null! Handle the case where setGradient(null)
   happened with the fixes from prior commit which crashed about.
   Closes surge-synthesizer#4495

2. Update the install-local-blah targets to be susrge xt named

3. Update the README with more of the cmake tips and tricks
@baconpaul baconpaul mentioned this issue May 6, 2021
Merged
baconpaul added a commit that referenced this issue May 6, 2021
@baconpaul
Fix Gradient Crash; Fix install-local targets; update README (#4496)
ea70b49 
1. can't remember a null! Handle the case where setGradient(null)
   happened with the fixes from prior commit which crashed about.
   Closes #4495

2. Update the install-local-blah targets to be susrge xt named

3. Update the README with more of the cmake tips and tricks
@bobobo199733 bobobo199733 mentioned this issue Mar 5, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Report Item submitted using the Bug Report template Linux Issues which only occur on Linux
Projects
None yet
Milestone
Surge XT 1.0
Development

Successfully merging a pull request may close this issue.

Fix Gradient Crash; Fix install-local targets; update README baconpaul/surge
2 participants
@mkruselj @baconpaul