linux-kernel - Re: [PATCH] sound: stm32: replace "%p" with "%pK"

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: <s5hpo00quk4.wl-tiwai@suse.de>
Date:   Fri, 06 Jul 2018 16:59:23 +0200
From:   Takashi Iwai <tiwai@...e.de>
To:     "Benjamin Gaignard" <benjamin.gaignard@...aro.org>
Cc:     <lgirdwood@...il.com>, <mcoquelin.stm32@...il.com>,
        <broonie@...nel.org>, <perex@...ex.cz>, <alexandre.torgue@...com>,
        <arnaud.pouliquen@...com>, <alsa-devel@...a-project.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        "Benjamin Gaignard" <benjamin.gaignard@...com>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] sound: stm32: replace "%p" with "%pK"

On Fri, 06 Jul 2018 15:07:03 +0200,
Benjamin Gaignard wrote:
> 
> The format specifier "%p" can leak kernel addresses.
> Use "%pK" instead.
> 
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@...com>

The code change itself is OK (apart from the subject prefix to be
"ASoC").  But do we really want to patch manually every single place
like this?  There are thousands of files using '%p'.


thanks,

Takashi