View Issue Details

IDProjectCategoryView StatusLast Update
0001266OpenMPT[All Projects] Plugins / VSTpublic2019-10-11 18:06
Reporterherodotas Assigned To 
PrioritynormalSeveritycrashReproducibilityalways
Status newResolutionopen 
Platformx64OSWindowsOS Version7
Product VersionOpenMPT 1.29.00.* (current testing) 
Target VersionFixed in Version 
Summary0001266: Unloading SynthEdit plugins causes crash if another SynthEdit plugin is loaded afterwards
Description

Hi,
After I try to change loaded SynthEdit x64 plugin to another, OPENMPT always crashes.

Steps To Reproduce
  1. Load first SynthEdit plugin, open plugin editor
  2. Unload the plugin
  3. Load another SynthEdit plugin, open editor
  4. Observe crash
TagsNo tags attached.
Has the bug occurred in previous versions?
Tested code revision (in case you know it)

Activities

herodotas

herodotas

2019-09-28 10:59

reporter  

OpenMPT Crash Files.rar (37,415 bytes)
Saga Musix

Saga Musix

2019-09-28 11:15

administrator   ~0004095

The crash dumps themselves are not too revealing so please provide some information which plugins caused the crash.

herodotas

herodotas

2019-09-28 14:49

reporter   ~0004096

Any x64 synthedit plugin. In my case: Kx Polymod, Aegis, Sk7, Synlay, Lynx.

Saga Musix

Saga Musix

2019-09-28 19:10

administrator   ~0004097

Here the plugins only crash here when trying to open the editor after having opened another SynthEdit plugin's editor. The crash happens deep inside effEditOpen so I doubt there is something we can do about this. However, forcing the plugins to run inside the plugin bridge circumvents the problem.

Saga Musix

Saga Musix

2019-09-28 19:39

administrator   ~0004098

I contacted the SynthEdit author as the issue even happened when I created a SynthEdit plugin that consisted of nothing but a knob.

Saga Musix

Saga Musix

2019-10-11 16:16

administrator   ~0004107

I got no reply from Jeff yet, but I was able to verify that not calling FreeLibrary when unloading the plugin prevents the issue from happening. Apparently Reaper even has an option which I guess does exactly that. While it would fix the problem, not calling FreeLibrary would of course be a rather dirty workaround that would potentially prevent memory used by unloaded plugins from being reused.

Saga Musix

Saga Musix

2019-10-11 17:10

administrator   ~0004108

I confirmed that the same crash happens in Reaper if you enable "Allow complete unload of VST-plug-ins", so it's not an OpenMPT-specific bug. This really is something that must be fixed in SynthEdit. Maybe we can add an option to OpenMPT to never unload plugins, but I don't think this should be enabled by default.

Issue History

Date Modified Username Field Change
2019-09-28 10:59 herodotas New Issue
2019-09-28 10:59 herodotas File Added: OpenMPT Crash Files.rar
2019-09-28 11:15 Saga Musix Note Added: 0004095
2019-09-28 14:49 herodotas Note Added: 0004096
2019-09-28 19:10 Saga Musix Note Added: 0004097
2019-09-28 19:39 Saga Musix Note Added: 0004098
2019-10-11 16:16 Saga Musix Note Added: 0004107
2019-10-11 17:10 Saga Musix Note Added: 0004108
2019-10-11 18:06 Saga Musix Summary SynthEdit plugins crash OpenMpt => Unloading SynthEdit plugins cause crash if another SynthEdit plugin is loaded
2019-10-11 18:06 Saga Musix Summary Unloading SynthEdit plugins cause crash if another SynthEdit plugin is loaded => Unloading SynthEdit plugins causes crash if another SynthEdit plugin is loaded
2019-10-11 18:06 Saga Musix Summary Unloading SynthEdit plugins causes crash if another SynthEdit plugin is loaded => Unloading SynthEdit plugins causes crash if another SynthEdit plugin is loaded afterwards
2019-10-11 18:06 Saga Musix Steps to Reproduce Updated View Revisions