commit:     ca6691d526b69992959fdcc342acc684a5003ae2
Author:     Brian Evans <grknight <AT> tuffmail <DOT> com>
AuthorDate: Thu Apr 10 15:27:58 2014 +0000
Commit:     Brian Evans <grknight <AT> lavabit <DOT> com>
CommitDate: Thu Apr 10 15:27:58 2014 +0000
URL:        
http://git.overlays.gentoo.org/gitweb/?p=proj/mysql-extras.git;a=commit;h=ca6691d5

Backport tzinfo fix from MariaDB. Bug 491176

Signed-off-by: Brian Evans <grknight <AT> tuffmail.com>

---
 00000_index.txt                      |   6 ++
 20008_all_mysql-tzinfo-symlink.patch | 103 +++++++++++++++++++++++++++++++++++
 2 files changed, 109 insertions(+)

diff --git a/00000_index.txt b/00000_index.txt
index 94ed5c3..17cac9f 100644
--- a/00000_index.txt
+++ b/00000_index.txt
@@ -1635,3 +1635,9 @@
 @pn mysql
 @pn percona-server
 @@ Remove -Werror from USE="debug" builds
+
+@patch 20008_all_mysql-tzinfo-symlink.patch
+@ver 5.05.36.00 to 5.99.99.99
+@pn mysql
+@pn percona-server
+@@ Backport tzinfo symlink fix from MariaDB bug 491176

diff --git a/20008_all_mysql-tzinfo-symlink.patch 
b/20008_all_mysql-tzinfo-symlink.patch
new file mode 100644
index 0000000..5b7a524
--- /dev/null
+++ b/20008_all_mysql-tzinfo-symlink.patch
@@ -0,0 +1,103 @@
+=== modified file 'sql/tztime.cc'
+--- sql/tztime.cc      2013-05-07 11:05:09 +0000
++++ sql/tztime.cc      2013-11-13 15:16:35 +0000
+@@ -2494,7 +2494,7 @@
+ 
+ */
+ my_bool
+-scan_tz_dir(char * name_end)
++scan_tz_dir(char * name_end, uint symlink_recursion_level)
+ {
+   MY_DIR *cur_dir;
+   char *name_end_tmp;
+@@ -2514,7 +2514,32 @@
+ 
+       if (MY_S_ISDIR(cur_dir->dir_entry[i].mystat->st_mode))
+       {
+-        if (scan_tz_dir(name_end_tmp))
++        my_bool is_symlink;
++        if ((is_symlink= my_is_symlink(fullname)) &&
++            symlink_recursion_level > 0)
++        {
++          /*
++            The timezone definition data in some Linux distributions
++             (e.g. the "timezone-data-2013f" package in Gentoo)
++            may have synlimks like:
++              /usr/share/zoneinfo/posix/ -> /usr/share/zoneinfo/,
++            so the same timezone files are available under two names
++            (e.g. "CET" and "posix/CET").
++
++            We allow one level of symlink recursion for backward
++            compatibility with earlier timezone data packages that have
++            duplicate copies of the same timezone files inside the root
++            directory and the "posix" subdirectory (instead of symlinking).
++            This makes "posix/CET" still available, but helps to avoid
++            following such symlinks infinitely:
++              /usr/share/zoneinfo/posix/posix/posix/.../posix/
++          */
++          fflush(stdout);
++          fprintf(stderr, "Warning: Skipping directory '%s': "
++                          "to avoid infinite symlink recursion.\n", fullname);
++          continue;
++        }
++        if (scan_tz_dir(name_end_tmp, symlink_recursion_level + is_symlink))
+         {
+           my_dirend(cur_dir);
+           return 1;
+@@ -2526,14 +2551,20 @@
+         if (!tz_load(fullname, &tz_info, &tz_storage))
+           print_tz_as_sql(root_name_end + 1, &tz_info);
+         else
++        {
++          fflush(stdout);
+           fprintf(stderr,
+                   "Warning: Unable to load '%s' as time zone. Skipping it.\n",
+                   fullname);
++        }
+         free_root(&tz_storage, MYF(0));
+       }
+       else
++      {
++        fflush(stdout);
+         fprintf(stderr, "Warning: '%s' is not regular file or directory\n",
+                 fullname);
++      }
+     }
+   }
+ 
+@@ -2566,8 +2597,9 @@
+     printf("TRUNCATE TABLE time_zone_transition;\n");
+     printf("TRUNCATE TABLE time_zone_transition_type;\n");
+ 
+-    if (scan_tz_dir(root_name_end))
++    if (scan_tz_dir(root_name_end, 0))
+     {
++      fflush(stdout);
+       fprintf(stderr, "There were fatal errors during processing "
+                       "of zoneinfo directory\n");
+       return 1;
+@@ -2586,6 +2618,7 @@
+     {
+       if (tz_load(argv[2], &tz_info, &tz_storage))
+       {
++        fflush(stdout);
+         fprintf(stderr, "Problems with zoneinfo file '%s'\n", argv[2]);
+         return 1;
+       }
+@@ -2595,6 +2628,7 @@
+     {
+       if (tz_load(argv[1], &tz_info, &tz_storage))
+       {
++        fflush(stdout);
+         fprintf(stderr, "Problems with zoneinfo file '%s'\n", argv[2]);
+         return 1;
+       }
+@@ -2604,6 +2638,7 @@
+     free_root(&tz_storage, MYF(0));
+   }
+ 
++  my_end(0);
+   return 0;
+ }
+ 
+

Reply via email to