Opened 15 years ago
Last modified 14 years ago
#1685 new defect
ladspd sample freq parameter and crash fix
| Reported by: | Owned by: | reimar | |
|---|---|---|---|
| Priority: | normal | Component: | af |
| Version: | HEAD | Severity: | normal |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Reproduced by developer: | no | Analyzed by developer: | no |
Description
Hi!
I tried to use highpass_iir_1890 filter and found two bugs in af_ladspa.c.
1) It checks parameter boundaries, but doesn't care about LADSPA_IS_HINT_SAMPLE_RATE. So, currently it's impossible to use any plugin which uses this HINT with sane parameters (eg. cutoff frequencies), since values are not multiplied by sample rate. Since the sample rate is not known at this stage in af_ladspa.c I disabled the boundary check in this case and output a warning.
2) mplayer crashes with SIGSEGV because it creates two instances of this mono filter for stereo channels, but it only has a single "activated" flag and so only the first instance gets activated properly. The second one crashes in the moment it is run().
Every plugin with defined activate()-callback and multiple instances will crash.
Regards, Wolfgang Breyha
Attachments (1)
Change History (3)
by , 15 years ago
| Attachment: | af_ladspa_c_fixes.patch added |
|---|
fix for both boundary check and crash