Ticket #683 (reopened defect)
fsodeviced shouldn't fail when loading alsa scenarios
| Reported by: | jama | Owned by: | morphis |
|---|---|---|---|
| Priority: | major | Milestone: | cornucopia-0.12 |
| Component: | cornucopia/fsodevice | Version: | 0.11 |
| Keywords: | Cc: |
Description
Currently when fsodeviced is loading alsa scenario and it's not valid (we have long history of alsa control renames and removals with different kernels), then whole fsodeviced dies (so e.g. user cannot use wifi to download better scenarios or beg/google for help).
2012-05-30T19:35:33.143755Z [DEBUG] fsodeviced : Found scenario 'voiphandset' - main volume = 0 fsodeviced: control.c:1680: snd_ctl_elem_list_get_id: Assertion `idx < obj->used' failed.
Much better behavior would be to write ERROR in log (saying which scenario file and which line is invalid) and then ignore this whole scenario or just that line and continue loading other scenarios and plugins etc. Don't let fsodeviced die, we need him! :).
Change History
comment:2 Changed 11 months ago by morphis
- Milestone changed from cornucopia-0.12 to cornucopia-0.13
comment:3 Changed 11 months ago by morphis
- Owner changed from mickey to morphis
- Status changed from new to accepted
comment:4 Changed 11 months ago by morphis
- Status changed from accepted to closed
- Resolution set to fixed
- Milestone changed from cornucopia-0.13 to cornucopia-0.12
comment:5 Changed 9 months ago by jama
IIRC this worked when I was testing it with fso-autorev, but now with 0.12.0-rc2 I get fatal error again :/
I've checked that 6e51e7eff5b088d2e995d04412bc0e8b39a74368 is in 0.12 branch so it should be also in rc2 :/.
2000-01-17T13:30:35.766403Z [DEBUG] HardwareAudioManager? <|>: Registering alsa (PlayerLibAlsa?) to handle format wav 2000-01-17T13:30:35.801363Z [DEBUG] RouterLibAlsa? <>: Found scenario 'voiphandset' - main volume = 0 fsodeviced: control.c:1680: snd_ctl_elem_list_get_id: Assertion `idx < obj->used' failed.
