Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 1 Aug 2019 13:14:43 -0700
From:   Kees Cook <keescook@chromium.org>
To:     Robin Lindner <robin.lindner1@t-online.de>
Cc:     re.emese@gmail.com, kernel-hardening@lists.openwall.com,
        Alexander Popov <alex.popov@linux.com>,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Documentation patch (gcc-plugins kernel)
Message-ID: <201908011311.A06FB03C6C@keescook>
References: <ebb6d995-a356-bc01-074b-6154a283e299@t-online.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <ebb6d995-a356-bc01-074b-6154a283e299@t-online.de>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, Aug 01, 2019 at 09:30:58AM +0200, Robin Lindner wrote:
> Cleaned documentation comment up. I removed the "TODO" because it was very old.

Hi, please send these patches "normally" (cc maintainers, include lkml,
inline not attachments, etc):
https://www.kernel.org/doc/html/latest/process/submitting-patches.html

> ---
>  scripts/gcc-plugins/stackleak_plugin.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/scripts/gcc-plugins/stackleak_plugin.c b/scripts/gcc-plugins/stackleak_plugin.c
> index dbd37460c573e..d8ba12c3bb238 100644
> --- a/scripts/gcc-plugins/stackleak_plugin.c
> +++ b/scripts/gcc-plugins/stackleak_plugin.c
> @@ -144,8 +144,6 @@ static unsigned int stackleak_instrument_execute(void)
>  	 *
>  	 * Case in point: native_save_fl on amd64 when optimized for size
>  	 * clobbers rdx if it were instrumented here.
> -	 *
> -	 * TODO: any more special cases?
>  	 */
>  	if (is_leaf &&
>  	    !TREE_PUBLIC(current_function_decl) &&

As to the content of the patch, let's also CC Alexander...

Are there no more special cases?

-- 
Kees Cook