diff options
author | José Bollo <jose.bollo@iot.bzh> | 2019-03-29 09:26:29 +0100 |
---|---|---|
committer | José Bollo <jose.bollo@iot.bzh> | 2019-04-05 10:39:31 +0200 |
commit | 966189b7a6f0d4288c4c68a7c6b024577bf6b724 (patch) | |
tree | 408c6c774b143faf969694478239d3b771688241 /mixer-binding | |
parent | b6aec989e0620322250b033bf339db7868d117d3 (diff) |
Fix a SIGSEGV during initialization
During the initialisation of 4a-softmixer, when the function
'ProcessOneAvirtSubdev' fails, the function 'AttachOneLoop'
was crashing because it tried to call 'snd_ctl_close' with
a NULL pointer.
This change fixes it. A test is introduced because depending
on the kind of loop, the order of initialization of control
versus subdevs differs.
At the same time, some gotos are improved to avoid a dummy
test -this are also virtual fixes-.
Also a tiny refactor of the 3 times declared variable 'subdev'
is expected to improve the readability.
Bug-AGL: SPEC-2287
Change-Id: Ie5231a1064090d772ea0ea7d86ad8c8d79b38d72
Signed-off-by: José Bollo <jose.bollo@iot.bzh>
Diffstat (limited to 'mixer-binding')
0 files changed, 0 insertions, 0 deletions