[PATCH] Fix crashes on invalid input in IBM gconv modules [BZ #17325]
Allan McRae
allan@archlinux.org
Mon Sep 1 00:33:00 GMT 2014
On 30/08/14 00:32, Florian Weimer wrote:
> On 08/29/2014 03:33 PM, Andreas Schwab wrote:
>> Florian Weimer <fweimer@redhat.com> writes:
>>
>>> + if (__builtin_expect (rp2->start == 0xffff, 0) \
>>
>> Please use either this
>>
>>> + if (__builtin_expect (ch == 0xffff, 0) \
>>
>> or this consistently.
>
> Oops, you are right. I went with the first variant, as in the original
> fix.
>
> Retested successfully.
>
> Allan, is this okay for master at this stage?
>
I am OK with this, once there is an ack to use __builtin_expect instead
of __glibc_likely.
I agree with you that the cleanup of all the __builtin_expect should
happen in a following patch so the change is minimal at this stage and
easy to backport.
Allan
More information about the Libc-alpha
mailing list