| Message ID | 20230512025410.52832-1-jiapeng.chong@linux.alibaba.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ASoC: SOF: topology: Fix missing error code in sof_route_load() | expand |
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); > - 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; > } >
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; }
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(-)