diff options
| author | Takashi Sakamoto <o-takashi@sakamocchi.jp> | 2014-07-22 23:11:03 +0900 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2014-07-22 16:21:17 +0200 |
| commit | 901401166464dc1875825235bb2541af31b4c384 (patch) | |
| tree | ee4daa19b6879f580000882de7c04f1c24dc3e13 /sound/firewire | |
| parent | 4da63c6fc426023d1a20e45508c47d7d68c6a53d (diff) | |
| download | op-kernel-dev-901401166464dc1875825235bb2541af31b4c384.zip op-kernel-dev-901401166464dc1875825235bb2541af31b4c384.tar.gz | |
ALSA: bebob: Fix a missing to unlock mutex in error handling case
In error handling case, special_clk_ctl_put() returns without unlock_mutex(),
therefore the mutex is still locked. This commit moves mutex_lock() after
the error handling case.
This commit is my solution for this post.
[PATCH -next] ALSA: bebob: Fix missing unlock on error in special_clk_ctl_put()
https://lkml.org/lkml/2014/7/20/12
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/firewire')
| -rw-r--r-- | sound/firewire/bebob/bebob_maudio.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/firewire/bebob/bebob_maudio.c b/sound/firewire/bebob/bebob_maudio.c index 6af50eb..fc470c6 100644 --- a/sound/firewire/bebob/bebob_maudio.c +++ b/sound/firewire/bebob/bebob_maudio.c @@ -379,12 +379,12 @@ static int special_clk_ctl_put(struct snd_kcontrol *kctl, struct special_params *params = bebob->maudio_special_quirk; int err, id; - mutex_lock(&bebob->mutex); - id = uval->value.enumerated.item[0]; if (id >= ARRAY_SIZE(special_clk_labels)) return 0; + mutex_lock(&bebob->mutex); + err = avc_maudio_set_special_clk(bebob, id, params->dig_in_fmt, params->dig_out_fmt, |
