[PATCH 1/4] __fdelt_chk: Removed range check

Florian Weimer fweimer@redhat.com
Fri Apr 12 12:49:00 GMT 2013


On 04/12/2013 07:33 AM, KOSAKI Motohiro wrote:
> +strong_alias (__fdelt_nochk, __fdelt_chk)
> +strong_alias (__fdelt_nochk, __fdelt_warn)

This change (which disables checking for existing compiled binaries) 
seems the wrong thing to do to me.

I tend to agree that it might make sense to make fd_set fortification 
optional, but it should be enabled by default.  Could you please change 
your patch so that it performs the checking by default, and preserves 
checking for applications which were compiled against pre-2.18 versions?

By the way, if you see crashes with Qt, we have a patch which replaces 
select with poll (qt-4.8-poll.patch in Fedora).   We tried to upstream 
it, but no luck so far.

-- 
Florian Weimer / Red Hat Product Security Team



More information about the Libc-alpha mailing list