Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <1511998718-4158-1-git-send-email-me@tobin.cc>
Date: Thu, 30 Nov 2017 10:38:36 +1100
From: "Tobin C. Harding" <me@...in.cc>
To: Linus Torvalds <torvalds@...ux-foundation.org>
Cc: "Tobin C. Harding" <me@...in.cc>,	David Laight <David.Laight@...lab.com>,
	Kees Cook <keescook@...omium.org>,
	"Jason A. Donenfeld" <Jason@...c4.com>,	Theodore Ts'o <tytso@....edu>,
	Paolo Bonzini <pbonzini@...hat.com>,	Tycho Andersen <tycho@...ho.ws>,
	"Roberts, William C" <william.c.roberts@...el.com>,
	Tejun Heo <tj@...nel.org>,	Jordan Glover <Golden_Miller83@...tonmail.ch>,
	Greg KH <gregkh@...uxfoundation.org>,	Petr Mladek <pmladek@...e.com>,
	Joe Perches <joe@...ches.com>,	Ian Campbell <ijc@...lion.org.uk>,
	Sergey Senozhatsky <sergey.senozhatsky@...il.com>,
	Catalin Marinas <catalin.marinas@....com>,
	Will Deacon <wilal.deacon@....com>,	Steven Rostedt <rostedt@...dmis.org>,
	Chris Fries <cfries@...gle.com>,	Dave Weinstein <olorin@...gle.com>,
	Daniel Micay <danielmicay@...il.com>,	Djalal Harouni <tixxdz@...il.com>,
	Radim Krcmár <rkrcmar@...hat.com>,
	David Miller <davem@...emloft.net>,
	Stephen Rothwell <sfr@...b.auug.org.au>,
	Andrey Ryabinin <aryabinin@...tuozzo.com>,
	Alexander Potapenko <glider@...gle.com>,
	Dmitry Vyukov <dvyukov@...gle.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	kernel-hardening@...ts.openwall.com,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Network Development <netdev@...r.kernel.org>
Subject: [PATCH 0/2] replace %pK with %p

We are now hashing addresses printed with %pK (when
kptr_restrict==0). Perhaps we can get rid of %pK (and kptr_restrict)
entirely. Instead of rushing ahead and doing so let's replace all printk
format strings that use %pK with %p.

It is a nice time to do this now while we are prepared for breakages
from applying the pointer hashing patch series.

The patch to remove kptr_restrict entirely should then be a non-event.

Second patch adds printk specifier %pz to display zeroed address. This
may be useful for fixing things that break during the fallout from
hashing and replacing %pK. We can always revert this patch if it turns
out to be worthless, right?

Patch 1 was created using

for file in $(git grep -l '%pK')
do
	perl -pi -e 's/%pK/%p/g' $file
done

thanks,
Tobin.

Tobin C. Harding (2):
  tree-wide: replace all users of %pK with %p
  printk: add specifier %pz, for zeroed address

 Documentation/printk-formats.txt                   | 11 +++
 arch/arm/mm/physaddr.c                             |  2 +-
 arch/arm64/mm/physaddr.c                           |  2 +-
 arch/mips/kernel/relocate.c                        | 10 +--
 arch/mips/kvm/mips.c                               |  2 +-
 arch/powerpc/perf/hv-24x7.c                        |  8 +--
 arch/s390/kvm/intercept.c                          |  2 +-
 arch/s390/kvm/kvm-s390.c                           | 10 +--
 arch/s390/kvm/trace-s390.h                         |  4 +-
 drivers/android/binder.c                           |  2 +-
 drivers/android/binder_alloc.c                     | 28 ++++----
 drivers/gpu/drm/exynos/exynos_drm_dsi.c            |  4 +-
 drivers/gpu/drm/exynos/exynos_drm_fimc.c           |  2 +-
 drivers/gpu/drm/exynos/exynos_drm_gem.c            |  2 +-
 drivers/gpu/drm/exynos/exynos_drm_gsc.c            |  2 +-
 drivers/gpu/drm/exynos/exynos_drm_ipp.c            | 22 +++---
 drivers/gpu/drm/exynos/exynos_drm_rotator.c        |  2 +-
 drivers/gpu/drm/i915/i915_debugfs.c                |  2 +-
 drivers/infiniband/hw/usnic/usnic_uiom.c           |  2 +-
 drivers/net/wireless/ath/ath10k/ahb.c              |  2 +-
 drivers/net/wireless/ath/ath10k/bmi.c              |  4 +-
 drivers/net/wireless/ath/ath10k/ce.c               |  4 +-
 drivers/net/wireless/ath/ath10k/core.c             |  4 +-
 drivers/net/wireless/ath/ath10k/htc.c              |  6 +-
 drivers/net/wireless/ath/ath10k/htt_rx.c           |  2 +-
 drivers/net/wireless/ath/ath10k/mac.c              | 22 +++---
 drivers/net/wireless/ath/ath10k/pci.c              |  2 +-
 drivers/net/wireless/ath/ath10k/testmode.c         |  4 +-
 drivers/net/wireless/ath/ath10k/txrx.c             |  2 +-
 drivers/net/wireless/ath/ath10k/usb.c              |  4 +-
 drivers/net/wireless/ath/ath10k/wmi.c              |  4 +-
 drivers/spi/spi-loopback-test.c                    | 12 ++--
 drivers/staging/ccree/ssi_buffer_mgr.c             | 54 +++++++-------
 drivers/staging/ccree/ssi_cipher.c                 |  4 +-
 drivers/staging/ccree/ssi_hash.c                   | 30 ++++----
 .../interface/vchiq_arm/vchiq_2835_arm.c           |  6 +-
 .../vc04_services/interface/vchiq_arm/vchiq_arm.c  | 16 ++---
 .../vc04_services/interface/vchiq_arm/vchiq_core.c | 84 +++++++++++-----------
 .../interface/vchiq_arm/vchiq_kern_lib.c           |  4 +-
 drivers/usb/core/devio.c                           | 14 ++--
 drivers/usb/core/hcd.c                             |  4 +-
 drivers/usb/core/urb.c                             |  2 +-
 drivers/usb/dwc3/dwc3-st.c                         |  2 +-
 drivers/usb/dwc3/gadget.c                          |  4 +-
 include/linux/filter.h                             |  2 +-
 kernel/cgroup/debug.c                              |  8 +--
 kernel/module.c                                    |  2 +-
 kernel/time/timer_list.c                           |  4 +-
 lib/vsprintf.c                                     | 26 +++++--
 mm/vmalloc.c                                       |  4 +-
 net/atm/proc.c                                     |  4 +-
 net/bluetooth/af_bluetooth.c                       |  2 +-
 net/can/bcm.c                                      |  6 +-
 net/can/proc.c                                     |  4 +-
 net/ipv4/ping.c                                    |  2 +-
 net/ipv4/raw.c                                     |  2 +-
 net/ipv4/tcp_ipv4.c                                |  6 +-
 net/ipv4/udp.c                                     |  2 +-
 net/ipv6/datagram.c                                |  2 +-
 net/ipv6/tcp_ipv6.c                                |  6 +-
 net/key/af_key.c                                   |  2 +-
 net/netlink/af_netlink.c                           |  2 +-
 net/packet/af_packet.c                             |  2 +-
 net/phonet/socket.c                                |  2 +-
 net/unix/af_unix.c                                 |  2 +-
 sound/soc/bcm/cygnus-pcm.c                         |  2 +-
 66 files changed, 269 insertions(+), 240 deletions(-)

-- 
2.7.4

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.