linux-kernel - Re: [PATCH] btrfs: fixed an incorrect variable assignment

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20221007164249.GU13389@twin.jikos.cz>
Date:   Fri, 7 Oct 2022 18:42:49 +0200
From:   David Sterba <dsterba@...e.cz>
To:     liujing <liujing@...s.chinamobile.com>
Cc:     clm@...com, josef@...icpanda.com, dsterba@...e.com,
        linux-btrfs@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] btrfs: fixed an incorrect variable assignment

On Thu, Sep 15, 2022 at 11:11:49PM -0400, liujing wrote:
> In the btrfs_reclaim_bgs_work function, 
> there is an assignment of int ret =0, 
> but this assignment is not used in the following code, 
> so it can be defined as int ret.
> 
> Signed-off-by: liujing <liujing@...s.chinamobile.com>
> ---
>  fs/btrfs/block-group.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/btrfs/block-group.c b/fs/btrfs/block-group.c
> index c52b6e245b9a..a4c7fb423244 100644
> --- a/fs/btrfs/block-group.c
> +++ b/fs/btrfs/block-group.c
> @@ -1571,7 +1571,7 @@ void btrfs_reclaim_bgs_work(struct work_struct *work)
>  	list_sort(NULL, &fs_info->reclaim_bgs, reclaim_bgs_cmp);
>  	while (!list_empty(&fs_info->reclaim_bgs)) {
>  		u64 zone_unusable;
> -		int ret = 0;
> +		int ret;

I'm not sure we need to fix that, is it fixing some warning? Also please
rephrase the subject, it's not 'incorrect', the code works as expected
but the initial value is not used.