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] Fix BZ 18985 out of bounds access in strftime


On 20 Sep 2015 16:16, Paul Pluzhnikov wrote:
> On Sun, Sep 20, 2015 at 4:10 PM, Mike Frysinger wrote:
> > buf = INVALID<H/tm_hour=1024> 1012
> 
> It's fairly trivial to modify my patch to show "INVALID tm_wday" instead of "?".

using a const string is fine.  better than just "?" imo.

> Doing that for tm_hour is just a bit more involved, though not that
> much. But it seems that that would be something Paul E. *doesn't* want
> in his "IST-2IDT,M3.4.4/26,M10.5.0" example.

i'm not sure that's readily obvious that there's a failure.  seems like it's
easy to pass on to other layers which also might not notice until it's much
too late.
-mike

Attachment: signature.asc
Description: Digital signature


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