staging: greybus: codecs: Fix reference counter leak in error handling
[ Upstream commit5.4-rM2-2.2.x-imx-squashed3952659a61
] gb_pm_runtime_get_sync has increased the usage counter of the device here. Forgetting to call gb_pm_runtime_put_noidle will result in usage counter leak in the error branch of (gbcodec_hw_params and gbcodec_prepare). We fixed it by adding it. Fixes:c388ae7696
("greybus: audio: Update pm runtime support in dai_ops callback") Signed-off-by: Zhang Qilong <zhangqilong3@huawei.com> Link: https://lore.kernel.org/r/20201109131347.1725288-2-zhangqilong3@huawei.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
parent
5daf659fdf
commit
aa1d8b9594
|
@ -489,6 +489,7 @@ static int gbcodec_hw_params(struct snd_pcm_substream *substream,
|
|||
if (ret) {
|
||||
dev_err_ratelimited(dai->dev, "%d: Error during set_config\n",
|
||||
ret);
|
||||
gb_pm_runtime_put_noidle(bundle);
|
||||
mutex_unlock(&codec->lock);
|
||||
return ret;
|
||||
}
|
||||
|
@ -565,6 +566,7 @@ static int gbcodec_prepare(struct snd_pcm_substream *substream,
|
|||
break;
|
||||
}
|
||||
if (ret) {
|
||||
gb_pm_runtime_put_noidle(bundle);
|
||||
mutex_unlock(&codec->lock);
|
||||
dev_err_ratelimited(dai->dev, "set_data_size failed:%d\n",
|
||||
ret);
|
||||
|
|
Loading…
Reference in New Issue