[PATCH] mm/damon/sysfs: Fix missing error code in damon_sysfs_attrs_add_dirs()

[PATCH] mm/damon/sysfs: Fix missing error code in damon_sysfs_attrs_add_dirs()

[Jiapeng Chong]

The error code is missing in this code scenario, add the error code
'-ENOMEM' to the return value 'err'.

Eliminate the follow smatch warning:

mm/damon/sysfs.c:1647 damon_sysfs_attrs_add_dirs() warn: missing error
code 'err'.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
 mm/damon/sysfs.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/mm/damon/sysfs.c b/mm/damon/sysfs.c
index 32a9d21c0db5..0eca71163924 100644
--- a/mm/damon/sysfs.c
+++ b/mm/damon/sysfs.c
@@ -1643,8 +1643,10 @@ static int damon_sysfs_attrs_add_dirs(struct damon_sysfs_attrs *attrs)
 	attrs->intervals = intervals;
 
 	nr_regions_range = damon_sysfs_ul_range_alloc(10, 1000);
-	if (!nr_regions_range)
+	if (!nr_regions_range) {
+		err = -ENOMEM;
 		goto put_intervals_out;
+	}
 
 	err = kobject_init_and_add(&nr_regions_range->kobj,
 			&damon_sysfs_ul_range_ktype, &attrs->kobj,
-- 
2.20.1.7.g153144c

Re: [PATCH] mm/damon/sysfs: Fix missing error code in damon_sysfs_attrs_add_dirs()

[SeongJae Park]

On Wed, 2 Mar 2022 19:11:20 +0800 Jiapeng Chong <jiapeng.chong@linux.alibaba.com> wrote:

> The error code is missing in this code scenario, add the error code
> '-ENOMEM' to the return value 'err'.
> 
> Eliminate the follow smatch warning:
> 
> mm/damon/sysfs.c:1647 damon_sysfs_attrs_add_dirs() warn: missing error
> code 'err'.
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>

Good eye, thank you!

Reviewed-by: SeongJae Park <sj@kernel.org>

Andrew, please fold this patch into
mm-damon-implement-a-minimal-stub-for-sysfs-based-damon-interface.patch[1].

[1] https://www.ozlabs.org/~akpm/mmotm/broken-out/mm-damon-implement-a-minimal-stub-for-sysfs-based-damon-interface.patch


Thanks,
SJ

> ---
>  mm/damon/sysfs.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/mm/damon/sysfs.c b/mm/damon/sysfs.c
> index 32a9d21c0db5..0eca71163924 100644
> --- a/mm/damon/sysfs.c
> +++ b/mm/damon/sysfs.c
> @@ -1643,8 +1643,10 @@ static int damon_sysfs_attrs_add_dirs(struct damon_sysfs_attrs *attrs)
>  	attrs->intervals = intervals;
>  
>  	nr_regions_range = damon_sysfs_ul_range_alloc(10, 1000);
> -	if (!nr_regions_range)
> +	if (!nr_regions_range) {
> +		err = -ENOMEM;
>  		goto put_intervals_out;
> +	}
>  
>  	err = kobject_init_and_add(&nr_regions_range->kobj,
>  			&damon_sysfs_ul_range_ktype, &attrs->kobj,
> -- 
> 2.20.1.7.g153144c