This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 3/6] Reinstantiate fd range check if and only if defined _STRICT_FD_SIZE_CHECK=1

From: Florian Weimer <fweimer at redhat dot com>
To: chrubis at suse dot cz
Cc: KOSAKI Motohiro <kosaki dot motohiro at gmail dot com>, libc-alpha <libc-alpha at sourceware dot org>
Date: Wed, 03 Apr 2013 12:00:04 +0200
Subject: Re: [PATCH 3/6] Reinstantiate fd range check if and only if defined _STRICT_FD_SIZE_CHECK=1
References: <1364570244-2088-1-git-send-email-kosaki dot motohiro at gmail dot com> <1364570244-2088-4-git-send-email-kosaki dot motohiro at gmail dot com> <515A9C80 dot 7020604 at redhat dot com> <CAHGf_=q5fK+u=hyuJOh45RE1SHXDyO_Z+mW4fKGDbb9b1hwbUw at mail dot gmail dot com> <515BCC5E dot 9010604 at redhat dot com> <20130403094821 dot GA1545 at rei>

On 04/03/2013 11:48 AM, chrubis@suse.cz wrote:

Hi!

How so?  _FORTIFY_SOURCE is not enabled by default.  If you switch it
on, there are other (valid) uses which break.


That is simply not true. It's not turned on in glibc by default but
several build systems enable it system-wide. For example SUSE
buildservice.

Yes, and those of us who enable it deal with the resulting applicationfailures.

I really don't like adding additional _FORTIFY_SOURCE options. Enablingthis feature is already sufficiently complicated.


--
Florian Weimer / Red Hat Product Security Team

References:
- Re: [PATCH 3/6] Reinstantiate fd range check if and only if defined _STRICT_FD_SIZE_CHECK=1
  - From: Florian Weimer
- Re: [PATCH 3/6] Reinstantiate fd range check if and only if defined _STRICT_FD_SIZE_CHECK=1
  - From: KOSAKI Motohiro
- Re: [PATCH 3/6] Reinstantiate fd range check if and only if defined _STRICT_FD_SIZE_CHECK=1
  - From: Florian Weimer
- Re: [PATCH 3/6] Reinstantiate fd range check if and only if defined _STRICT_FD_SIZE_CHECK=1
  - From: chrubis

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]