[PATCH] ARM: traps: print un-hashed user pc on undefined instruction

Russell King (Oracle) linux at armlinux.org.uk
Thu Jul 28 08:23:17 PDT 2022


On Wed, Jul 27, 2022 at 07:18:01PM +0300, Baruch Siach wrote:
> When user undefined instruction debug is enabled pc value is hashed like
> kernel pointers for security reason. But the security benefit of this
> hash is very limited because the code goes on to call __show_regs() that
> prints the plain pointer value. pc is a user pointer anyway, so the
> kernel does not leak anything. The only result is confusion about the
> difference between the pc value on the first printed line, and the value
> that __show_regs() prints.
> 
> Always print the plain value of pc.
> 
> Signed-off-by: Baruch Siach <baruch at tkos.co.il>

Looks sensible to me. Please drop it in the patch system, thanks.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!



More information about the linux-arm-kernel mailing list