This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v4] Ensure mktime sets errno on error (bug 23789)
- From: Albert ARIBAUD <albert dot aribaud at 3adev dot fr>
- To: Paul Eggert <eggert at cs dot ucla dot edu>
- Cc: libc-alpha at sourceware dot org, Gnulib bugs <bug-gnulib at gnu dot org>
- Date: Tue, 6 Nov 2018 21:41:47 +0100
- Subject: Re: [PATCH v4] Ensure mktime sets errno on error (bug 23789)
- References: <20181030111850.5322-1-albert.aribaud@3adev.fr> <9947f682-fc46-f2d6-4b07-536c03b9e657@cs.ucla.edu> <e9704996-c39a-d4cb-2a62-b0342924cdc5@cs.ucla.edu> <20181106064306.45eda0c2@athena>
Hi Paul,
On Tue, 6 Nov 2018 06:43:06 +0100, Albert ARIBAUD
<albert.aribaud@3adev.fr> wrote :
> Hi Paul,
>
> On Sun, 4 Nov 2018 23:54:59 -0800, Paul Eggert <eggert@cs.ucla.edu>
> wrote :
>
> > Paul Eggert wrote:
> > > 3. Please construct a third patch containing your mktime test case for glibc,
> > > and we then apply that patch to glibc.
> >
> > I looked at that test case and found some issues with it, e.g., it assumed a
> > particular time_t width in some cases and assumed a particular error number in
> > others. Attached is a revised test case that should fix the issues. For
> > convenience I'm also attaching the same glibc code patch again.
>
> Apparently, with both your patches applied there are still paths which
> yield "mktime failed without setting errno" when make check is run for
> i686-linux-gnu. I'll go through the call path and see where it fails.
Issue is that __mktime_internal exited through
else if (--remaining_probes == 0)
return -1;
with errno never set.
Any idea why?
Cordialement,
Albert ARIBAUD
3ADEV