[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 4 Oct 2017 09:08:57 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: "Tobin C. Harding" <me@...in.cc>, Petr Mladek <pmladek@...e.com>, Joe
Perches <joe@...ches.com>, Ian Campbell <ijc@...lion.org.uk>, Sergey
Senozhatsky <sergey.senozhatsky@...il.com>,
kernel-hardening@...ts.openwall.com, linux-kernel@...r.kernel.org, Catalin
Marinas <catalin.marinas@....com>, Will Deacon <will.deacon@....com>,
William Roberts <william.c.roberts@...el.com>, Chris Fries
<cfries@...gle.com>, Dave Weinstein <olorin@...gle.com>
Subject: Re: [RFC V2 1/6] lib: vsprintf: additional
kernel pointer filtering options
On Wed, 4 Oct 2017 10:55:42 +0200
Greg KH <gregkh@...uxfoundation.org> wrote:
> On Sun, Oct 01, 2017 at 11:06:45AM +1100, Tobin C. Harding wrote:
> > Add the kptr_restrict setting of 3 which results in both
> > %p and %pK values being replaced by zeros.
> >
> > Add an additional %pP value inspired by the Grsecurity
> > option which explicitly whitelists pointers for output.
> >
> > Amend scripts/checkpatch.pl to handle %pP.
> >
> > This patch is based on work by William Roberts
> > <william.c.roberts@...el.com>
> >
> > Signed-off-by: Tobin C. Harding <me@...in.cc>
>
> Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Greg,
I'm curious to why you are adding a "Signed-off-by"? That usually means
that you either help author it, or it is going through your tree. I've
never seen a "Signed-off-by" in passing. That's usually a "Acked-by" or
"Reviewed-by".
In other words, I was told that a "Signed-off-by" should never be added
by anyone but the one who writes it. The original author adds it to the
patch they send, and the maintainer adds it when they pull in that
patch. But sending out a "Signed-off-by" like this, to be added, would
require someone else writing it for you. Which I was told was a no-no.
-- Steve
Powered by blists - more mailing lists
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
