diff options
author | José Bollo <jose.bollo@iot.bzh> | 2019-03-29 09:26:29 +0100 |
---|---|---|
committer | José Bollo <jose.bollo@iot.bzh> | 2019-03-29 09:40:26 +0000 |
commit | 074e6871959279c4aed3e162f21f677e08338b3a (patch) | |
tree | b7031cd5f616deaf51e7f25c78106630f423607c /mixer-binding | |
parent | 943f34b1d319e0ddc157dcfd85acde628f9d2ead (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