check LocationData for null on access of vibdelay

Wohlstand / libOPNMIDI

A Software MIDI Synthesizer library with OPN2 (YM2612) emulator

GNU Lesser General Public License v3.0

96 stars 6 forks source link

Closed jpcima closed 6 years ago

jpcima commented 6 years ago

a suggestion to fix a null issue mentioned in #15

Wohlstand commented 6 years ago

Another crash, damn! default And yeah, I made the draft for a unit test to validate that stuff better, now I'll try to write various cases into it...

Wohlstand commented 6 years ago

Okay, just now I have posted a working unit test which reproduces a crash at me: https://github.com/Wohlstand/libOPNMIDI/commit/3414d9da907abece64a5a93de89b658ae8aa9c92 You can try it out and even extend it with something more to polish the result. For now I must go to meet my friend, so, I'll continue everything later. Cia :wink: :fox_face:

jpcima commented 6 years ago

NP I know the problem I'll fix it, no worry