[Bug libc/11478] New: __tzset_parse_tz doesn't conform to POSIX

[Bug libc/11478] New: __tzset_parse_tz doesn't conform to POSIX

[suzuki at in dot ibm dot com]

The __tzset_parse_tz fails to parse the valid string formats as described by POSIX.

This was introduced with the following commit.

http://sourceware.org/git/?p=glibc.git;a=commit;h=82780cbee540bc7e61de6eb165d0f3ec7aa9e549

$ cat tzset-minimal.c 
#include <time.h>

extern char* tzname[];


main()
{

	tzset();
	printf ("std tzname is  %s\n", *tzname[0]?tzname[0]:"NULL");
	return 0;
}
$ TZ="A?BC0" ./tzset-minimal
std tzname is  NULL

-- 
           Summary: __tzset_parse_tz doesn't conform to POSIX
           Product: glibc
           Version: 2.9
            Status: NEW
          Severity: normal
          Priority: P2
         Component: libc
        AssignedTo: drepper at redhat dot com
        ReportedBy: suzuki at in dot ibm dot com
                CC: glibc-bugs at sources dot redhat dot com
 GCC build triplet: all
  GCC host triplet: all
GCC target triplet: all


http://sourceware.org/bugzilla/show_bug.cgi?id=11478

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11478] __tzset_parse_tz doesn't conform to POSIX

[schwab at linux-m68k dot org]

------- Additional Comments From schwab at linux-m68k dot org  2010-04-08 11:49 -------
POSIX does not specify the interpretation of this time zone string.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11478

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11478] __tzset_parse_tz doesn't conform to POSIX

[suzuki at in dot ibm dot com]

------- Additional Comments From suzuki at in dot ibm dot com  2010-04-08 11:59 -------
http://www.opengroup.org/onlinepubs/007908775/xbd/envvar.html

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11478

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11478] __tzset_parse_tz doesn't conform to POSIX

[suzuki at in dot ibm dot com]

------- Additional Comments From suzuki at in dot ibm dot com  2010-04-08 12:02 -------
The tzset() page points to the page for environment variables.

http://www.opengroup.org/onlinepubs/007908775/xsh/tzset.html ->

http://www.opengroup.org/onlinepubs/007908775/xbd/envvar.html

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11478

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11478] __tzset_parse_tz doesn't conform to POSIX

[drepper at redhat dot com]

------- Additional Comments From drepper at redhat dot com  2010-04-08 12:12 -------
(In reply to comment #3)
> The tzset() page points to the page for environment variables.
> 
> http://www.opengroup.org/onlinepubs/007908775/xsh/tzset.html ->
> 
> http://www.opengroup.org/onlinepubs/007908775/xbd/envvar.html

And?  The format is specified as

     std offset dst offset, rule

where std is either

  <[a-zA-Z0-9+-]{3,TZNAME_MAX}>

or

  [a-zA-Z]{3,TZNAME_MAX}

Nowhere a question mark is allowed.  That form is just invalid and the
implementation is correct to reject it.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |INVALID


http://sourceware.org/bugzilla/show_bug.cgi?id=11478

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/11478] __tzset_parse_tz doesn't conform to POSIX

[suzuki at in dot ibm dot com]

------- Additional Comments From suzuki at in dot ibm dot com  2010-04-08 12:42 -------
You are right. POSIX doesn't say it can be any character. It clearly says it
should be alphanumeric  in < > or alphabetic. It is the SUSv3 which says it can
be any character.

[ snip ]

Where:

std and dst

    Indicates no less than three, nor more than {TZNAME_MAX}, bytes that are the
designation for the standard ( std ) or the alternative ( dst - such as Daylight
Savings Time) timezone. Only std is required; if dst is missing, then the
alternative time does not apply in this locale. Upper- and lower-case letters
are explicitly allowed. Any graphic characters except a leading colon (:) or
digits, the comma (,), the minus (-), the plus (+), and the null character are
permitted to appear in these fields, but their meaning is unspecified. 


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=11478

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.