gtimezone: consider a leading : in TZ environment variable
authorGiovanni Campagna <gcampagna@src.gnome.org>
Wed, 16 Nov 2011 23:52:59 +0000 (00:52 +0100)
committerGiovanni Campagna <gcampagna@src.gnome.org>
Mon, 6 Feb 2012 16:26:30 +0000 (17:26 +0100)
When set to represent a zoneinfo file, TZ may start with :, therefore
glib needs to check it and ignore the first char when building the
resulting filename, or it won't be found.
Also, the path could be absolute, in which case it is wrong to
append /usr/share/timezone

https://bugzilla.gnome.org/show_bug.cgi?id=664237

glib/gtimezone.c

index 2785d22..8195106 100644 (file)
@@ -368,6 +368,10 @@ g_time_zone_new (const gchar *identifier)
         {
           gchar *filename;
 
+         /* identifier can be a relative or absolute path name;
+            if relative, it is interpreted starting from /usr/share/timezone
+            while the POSIX standard says it should start with :,
+            glibc allows both syntaxes, so we should too */
           if (identifier != NULL)
             {
               const gchar *tzdir;
@@ -376,7 +380,13 @@ g_time_zone_new (const gchar *identifier)
               if (tzdir == NULL)
                 tzdir = "/usr/share/zoneinfo";
 
-              filename = g_build_filename (tzdir, identifier, NULL);
+             if (*identifier == ':')
+               identifier ++;
+
+             if (g_path_is_absolute (identifier))
+               filename = g_strdup (identifier);
+             else
+               filename = g_build_filename (tzdir, identifier, NULL);
             }
           else
             filename = g_strdup ("/etc/localtime");