Re: CVS commit: src/lib/libc/time

[Aleksej Saushev <asau%inbox.ru@localhost>]

"Brian Ginsbach" <ginsbach%netbsd.org@localhost>
writes:

> Module Name:	src
> Committed By:	ginsbach
> Date:		Fri Oct 30 01:49:36 UTC 2015
>
> Modified Files:
> 	src/lib/libc/time: strptime.c
>
> Log Message:
> Reject timezone offsets more than 12 hours (east or west).

WTF?? There exist (or existed) zones +13 and +14.


> To generate a diff of this commit:
> cvs rdiff -u -r1.51 -r1.52 src/lib/libc/time/strptime.c
>
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>
>
> Modified files:
>
>
> Index: src/lib/libc/time/strptime.c
> diff -u src/lib/libc/time/strptime.c:1.51 src/lib/libc/time/strptime.c:1.52
> --- src/lib/libc/time/strptime.c:1.51	Thu Oct 29 19:18:19 2015
> +++ src/lib/libc/time/strptime.c	Fri Oct 30 01:49:36 2015
> @@ -1,4 +1,4 @@
> -/*	$NetBSD: strptime.c,v 1.51 2015/10/29 19:18:19 christos Exp $	*/
> +/*	$NetBSD: strptime.c,v 1.52 2015/10/30 01:49:36 ginsbach Exp $	*/
>  
>  /*-
>   * Copyright (c) 1997, 1998, 2005, 2008 The NetBSD Foundation, Inc.
> @@ -31,7 +31,7 @@
>  
>  #include <sys/cdefs.h>
>  #if defined(LIBC_SCCS) && !defined(lint)
> -__RCSID("$NetBSD: strptime.c,v 1.51 2015/10/29 19:18:19 christos Exp $");
> +__RCSID("$NetBSD: strptime.c,v 1.52 2015/10/30 01:49:36 ginsbach Exp $");
>  #endif
>  
>  #include "namespace.h"
> @@ -626,6 +626,8 @@ loadzone:
>  			default:
>  				return NULL;
>  			}
> +			if (offs > (12 * SECSPERHOUR))
> +				return NULL;
>  			if (neg)
>  				offs = -offs;
>  			tm->tm_isdst = 0;	/* XXX */
>

-- 
HE CE3OH...