[Buildroot] [PATCH 2/2] toolchain/wrapper: extend paranoid check to -isystem
Arnout Vandecappelle
arnout at mind.be
Wed Aug 24 01:18:59 UTC 2016
On 17-08-16 16:42, Yann E. MORIN wrote:
> Some packages, like libbsd, use -isystem flags to provide so-called
> overrides to the system include files. In this particular case, this
> is used in a .pc file, then used by antoher package; pkgconf does not
> mangle this path; and eventually that other package ends up using
> /usr/include/bsd to search for headers.
>
> Our current toolchain wrapper is limited to looking fo -I and -L, so
> the paranoid check does not kick in.
>
> Extend the paranoid check to also look for the -isystem option.
While we're at it: -idirafter, -iprefix, -iwithprefix, -iwithprefixbefore,
-isysroot, -imultilib, -iquote.
And then there is -B, but if someone passes that, it's really broken :-) And
--sysroot, also interesting if that is passed. But I guess these things are
going a bit too far.
>
> Signed-off-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
> Cc: Thomas Petazzoni <thomas.petazzoni at free-electrons.com>
> Cc: Arnout Vandecappelle <arnout at mind.be>
> ---
> toolchain/toolchain-wrapper.c | 18 ++++++++++--------
> 1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/toolchain/toolchain-wrapper.c b/toolchain/toolchain-wrapper.c
> index b8b3cbe..8a9c3b3 100644
> --- a/toolchain/toolchain-wrapper.c
> +++ b/toolchain/toolchain-wrapper.c
> @@ -241,17 +241,19 @@ int main(int argc, char **argv)
> /* Check for unsafe library and header paths */
> for (i = 1; i < argc; i++) {
>
> - /* Skip options that do not start with -I and -L */
> - if (strncmp(argv[i], "-I", 2) && strncmp(argv[i], "-L", 2))
> + /* Skip options that do not start with -I, -isystem or -L */
> + if ( strncmp(argv[i], "-I", 2)
> + && strncmp(argv[i], "-L", 2)
> + && strcmp(argv[i], "-isystem"))
> continue;
>
> - /* We handle two cases: first the case where -I/-L and
> - * the path are separated by one space and therefore
> - * visible as two separate options, and then the case
> - * where they are stuck together forming one single
> - * option.
> + /* We handle two cases: first the case where -I/-L/-isystem
> + * and the path are separated by one space and therefore
> + * visible as two separate options, and then the case where
> + * they are stuck together forming one single option.
> + * -isystem is necessarily in the first case.
Unfortunately, that's not true. You can pass something like -isystemfoo and it
will add ./foo to the search path. Ain't gcc fun... I'll leave it as an exercise
to the reader to handle that case :-P
Regards,
Arnout
> */
> - if (argv[i][2] == '\0') {
> + if (argv[i][2] == '\0' || argv[i][1] == 'i') {
> i++;
> if (i == argc)
> continue;
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the buildroot
mailing list