Re: [patch 5/5] Add the -fstack-protector option to the CFLAGS

From: David Miller
Date: Fri Jul 28 2006 - 19:10:10 EST


From: Valdis.Kletnieks@xxxxxx
Date: Fri, 28 Jul 2006 19:05:40 -0400

> On Fri, 28 Jul 2006 20:48:31 +0200, Arjan van de Ven said:
> > On Fri, 2006-07-28 at 20:45 +0200, Andi Kleen wrote:
> > > > +ifdef CONFIG_CC_STACKPROTECTOR
> > > > +CFLAGS += $(call cc-ifversion, -lt, 0402, -fno-stack-protector)
> > > > +CFLAGS += $(call cc-ifversion, -ge, 0402, -fstack-protector)
> > >
> > > Why can't you just use the normal call cc-option for this?
> >
> > this requires gcc 4.2; cc-option is not useful for that.
>
> At least some things calling themselves 4.1.1 include it. On a Fedora

Your gcc-4.1.1 includes the -fstack-protector feature, but it might
not have the gcc bug fix necessary to make that feature work on the
kernel compile, which is why the version check is necessary.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/