[PATCH] ASoC: SOF: topology: Fix missing error code in sof_route_load()
The error code is missing in this code scenario, add the error code '-EINVAL' to the return value 'err'.
sound/soc/sof/topology.c:2060 sof_route_load() warn: missing error code 'ret'.
Reported-by: Abaci Robot abaci@linux.alibaba.com Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=4935 Signed-off-by: Jiapeng Chong jiapeng.chong@linux.alibaba.com --- sound/soc/sof/topology.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/sof/topology.c b/sound/soc/sof/topology.c index d3d536b0a8f5..3ca3a72f1805 100644 --- a/sound/soc/sof/topology.c +++ b/sound/soc/sof/topology.c @@ -2045,7 +2045,6 @@ static int sof_route_load(struct snd_soc_component *scomp, int index, if (!source_swidget) { dev_err(scomp->dev, "error: source %s not found\n", route->source); - ret = -EINVAL; goto err; }
@@ -2064,7 +2063,6 @@ static int sof_route_load(struct snd_soc_component *scomp, int index, if (!sink_swidget) { dev_err(scomp->dev, "error: sink %s not found\n", route->sink); - ret = -EINVAL; goto err; }
@@ -2087,6 +2085,8 @@ static int sof_route_load(struct snd_soc_component *scomp, int index, return 0; err: kfree(sroute); + ret = -EINVAL; + return ret; }
On 12/05/2023 05:54, Jiapeng Chong wrote:
The error code is missing in this code scenario, add the error code '-EINVAL' to the return value 'err'.
sound/soc/sof/topology.c:2060 sof_route_load() warn: missing error
code 'ret'.
Reported-by: Abaci Robot abaci@linux.alibaba.com
Robots don't necessarily understand code... The cases which this is aiming to fix are the cases when we ignore things, not failing.
Would it be possible to check the code and understand it before sending a patch?
Nack.
Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=4935 Signed-off-by: Jiapeng Chong jiapeng.chong@linux.alibaba.com
sound/soc/sof/topology.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/sound/soc/sof/topology.c b/sound/soc/sof/topology.c index d3d536b0a8f5..3ca3a72f1805 100644 --- a/sound/soc/sof/topology.c +++ b/sound/soc/sof/topology.c @@ -2045,7 +2045,6 @@ static int sof_route_load(struct
snd_soc_component *scomp, int index,
if (!source_swidget) { dev_err(scomp->dev, "error: source %s not found\n", route->source);
goto err; }ret = -EINVAL;
@@ -2064,7 +2063,6 @@ static int sof_route_load(struct
snd_soc_component *scomp, int index,
if (!sink_swidget) { dev_err(scomp->dev, "error: sink %s not found\n", route->sink);
goto err; }ret = -EINVAL;
@@ -2087,6 +2085,8 @@ static int sof_route_load(struct
snd_soc_component *scomp, int index,
return 0; err: kfree(sroute);
- ret = -EINVAL;
- return ret;
}
participants (2)
-
Jiapeng Chong
-
Péter Ujfalusi