linux-kernel - Re: [PATCH] tracing: Remove redundant assignment to variable ret'

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] [thread-next>] [day] [month] [year] [list]

Message-Id: <20220117074117.3770-1-lukas.bulwahn@gmail.com>
Date:   Mon, 17 Jan 2022 08:41:17 +0100
From:   Lukas Bulwahn <lukas.bulwahn@...il.com>
To:     ytcoode@...il.com
Cc:     linux-kernel@...r.kernel.org, mingo@...hat.com,
        rostedt@...dmis.org, Lukas Bulwahn <lukas.bulwahn@...il.com>
Subject: Re: [PATCH] tracing: Remove redundant assignment to variable ret'

Dear Yuntao,

when you consider removing dead-store assignments guided by some static
analyzer, you need to check if the code you are looking at is actually
missing an error-handling branch.

In this case, ftrace_process_locs() may return -ENOMEM, and the caller
needs to appropriately deal with this error return code. Your patch
does not change the code at all, i.e., the compiled object code is the
same as after the patch as before.

Think about how to deal appropriately with the -ENOMEM return in this
caller and submit a patch that implements the right error-handling
branch or argue in your commit message why that is not needed at all.

If you do not understand or cannot check such basic code properties for
dead-store assignments, it might be better to work on some other aspect
and area of the kernel repository. E.g., the kernel documentation build
also has a few warnings that deserve patches to be fixed.

Best regards,

Lukas