On Fri, Jan 13, 2017 at 05:01:01PM +0100, Nicolas Dichtel wrote:

Please, do not remove the email subject when you reply. I restore it to ease the thread follow-up.

I mentioned it to David, and he says it's because the long list of recipients is breaking his mailer. I've already posed the question about whether that's exploitable!

Le 13/01/2017 à 16:36, David Howells a écrit :

...

Nicolas Dichtel nicolas.dichtel@6wind.com wrote:

...

This header file is exported, thus move it to uapi.

Exported how?

It is listed in include/uapi/asm-generic/Kbuild.asm, which is included by arch/arm/include/uapi/asm/Kbuild.

We really should not be installing non-uapi header files to userland under _any_ circumstance - this to me sounds like a bug in kbuild.

The assumption is that headers outside of uapi directories are not part of the user visible API, and so can be freely modified - which in the presence of this bug is untrue.

However, as it's happening, and this header has been there since 2013 (commit 09096f6a0ee2 - "ARM: 7822/1: add workaround for ambiguous C99 stdint.h types") it's now well and truely part of the user API whether we intended it to be or not, so your patch looks to me like the correct thing to do.

I think it needs further evaluation to make sure kbuild isn't going to do something else silly, like subsitute include/asm-generic/types.h for the now missing arch/arm/include/asm/types.h

I wonder how many more headers are unintentionally exported.

... what a mess. :(