Ping
On Sat, Sep 10, 2016 at 06:54:48PM +1200, Carlin Bingham wrote:
> If gmtime_r(3) fails it should return NULL, it currently returns a
> pointer to the unitialised tm struct it was given.
>
>
> --
> Carlin
>
>
>
> Index: lib/libc/time/localtime.c
> ===================================================================
> RCS file: /cvs/src/lib/libc/time/localtime.c,v
> retrieving revision 1.58
> diff -u -p -u -r1.58 localtime.c
> --- lib/libc/time/localtime.c 14 Mar 2016 15:26:52 -0000 1.58
> +++ lib/libc/time/localtime.c 10 Sep 2016 06:06:24 -0000
> @@ -1350,8 +1350,7 @@ gmtsub(const time_t *timep, long offset,
> struct tm *
> gmtime_r(const time_t *timep, struct tm *p_tm)
> {
> - gmtsub(timep, 0L, p_tm);
> - return p_tm;
> + return gmtsub(timep, 0L, p_tm);
> }
> DEF_WEAK(gmtime_r);
>
>