DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D741E21795
Date: Mon, 15 Jan 2018 07:52:31 -0500
From: Steven Rostedt <rostedt@goodmis.org>
To: Dan Aloni <dan@kernelim.com>
Cc: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>,
        linux-kernel@vger.kernel.org, kernel-hardening@lists.openwall.com,
        Petr Mladek <pmladek@suse.com>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        Kees Cook <keescook@chromium.org>
Subject: Re: [PATCHv2 5/7] printk: allow kmsg to be encrypted using public
 key encryption
Message-ID: <20180115075231.0d435fec@gandalf.local.home>
In-Reply-To: <20180114080108.GA69022@gmail.com>
References: <20180113213441.52047-1-dan@kernelim.com>
        <20180113213441.52047-6-dan@kernelim.com>
        <20180114014801.GC1701@tigerII.localdomain>
        <20180114080108.GA69022@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org

On Sun, 14 Jan 2018 10:01:08 +0200
Dan Aloni <dan@kernelim.com> wrote:

> On Sun, Jan 14, 2018 at 10:48:01AM +0900, Sergey Senozhatsky wrote:
> > Ccing Kees, Peter, Andrew, Steven
> > 
> > On (01/13/18 23:34), Dan Aloni wrote:  
> > > This commit enables the kernel to encrypt the free-form text that
> > > is generated by printk() before it is brought up to `dmesg` in
> > > userspace.
> > > 
> > > The encryption is made using one of the trusted public keys which
> > > are kept built-in inside the kernel. These keys are presently
> > > also used for verifying kernel modules and userspace-supplied
> > > firmwares.  
> > 
> > OK, this is the first time I'm receiving it, yet it's v2 already.
> > I'm Cc-ed on only this particular patch, not the entire patch set;
> > so it's hard to tell what else is being touched and why, so I'm
> > going to start with the basic questions.  
> 
> Sorry, here the link to cover letter:
> 
>     https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1585442.html
> 
> I guess --cc-cover && --to-cover should be default.
> 
> > are you fixing the real problem? that's because you see unhashed
> > kernel pointers in dmesg or is there anything else?  
> 
> In brief, the problem is that any information leak has the potential
> to make exploitation easier. The changes include documentation for
> the feature, where more details are provided.
> 

I'm very skeptical that such an approach has much benefit. From the
email referenced above:

> I am not sure that desktop and power users would like to have their
> kernel message encrypted, but there are scenarios such as in mobile
> devices, where only the developers, makers of devices, may actually
> benefit from access to kernel prints messages, and the users may be
> more protected from exploits.

Do you have any backing from makers of such devices? I'd like to hear
from Google's Android team or whoever that would turn this feature on.

I would be hard pressed to add such a feature if it's never used.

-- Steve