]> git.ipfire.org Git - thirdparty/glibc.git/commitdiff
gconv: Use 64-bit interfaces in gconv_parseconfdir (bug 29583)
authorFlorian Weimer <fweimer@redhat.com>
Tue, 20 Sep 2022 10:12:43 +0000 (12:12 +0200)
committerFlorian Weimer <fweimer@redhat.com>
Wed, 21 Sep 2022 11:13:02 +0000 (13:13 +0200)
It's possible that inode numbers are outside the 32-bit range.
The existing code only handles the in-libc case correctly, and
still uses the legacy interfaces when building iconv.

Suggested-by: Helge Deller <deller@gmx.de>
(cherry picked from commit f97905f24631097af325d6a231093071c3077a5f)

NEWS
iconv/gconv_parseconfdir.h

diff --git a/NEWS b/NEWS
index 4a2531aa9729f07177d48c1ba33a6fbacd396ef3..1eaa415d543b38f5c7e039defd94b61d7009a1d8 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -117,6 +117,7 @@ The following bugs are resolved with this release:
   [29446] _dlopen now ignores dl_caller argument in static mode
   [29490] alpha: New __brk_call implementation is broken
   [29528] elf: Call __libc_early_init for reused namespaces
+  [29583] Use 64-bit interfaces in gconv_parseconfdir
 
 \f
 Version 2.34
index 79398a980cde84e3318f5d4a77c6c1cc2e8d0447..741cf7c67e36eccd027a34cdcbb319172dbadc41 100644 (file)
 # define isspace(__c) __isspace_l ((__c), _nl_C_locobj_ptr)
 # define asprintf __asprintf
 # define opendir __opendir
-# define readdir __readdir
+# define readdir64 __readdir64
 # define closedir __closedir
 # define mempcpy __mempcpy
-# define struct_stat struct __stat64_t64
-# define lstat __lstat64_time64
+# define struct_stat64 struct __stat64_t64
+# define lstat64 __lstat64_time64
 # define feof_unlocked __feof_unlocked
 #else
-# define struct_stat struct stat
+# define struct_stat64 struct stat64
 #endif
 
 /* Name of the file containing the module information in the directories
@@ -148,8 +148,8 @@ gconv_parseconfdir (const char *prefix, const char *dir, size_t dir_len)
   DIR *confdir = opendir (buf);
   if (confdir != NULL)
     {
-      struct dirent *ent;
-      while ((ent = readdir (confdir)) != NULL)
+      struct dirent64 *ent;
+      while ((ent = readdir64 (confdir)) != NULL)
        {
          if (ent->d_type != DT_REG && ent->d_type != DT_UNKNOWN)
            continue;
@@ -161,12 +161,12 @@ gconv_parseconfdir (const char *prefix, const char *dir, size_t dir_len)
              && strcmp (ent->d_name + len - strlen (suffix), suffix) == 0)
            {
              char *conf;
-             struct_stat st;
+             struct_stat64 st;
              if (asprintf (&conf, "%s/%s", buf, ent->d_name) < 0)
                continue;
 
              if (ent->d_type != DT_UNKNOWN
-                 || (lstat (conf, &st) != -1 && S_ISREG (st.st_mode)))
+                 || (lstat64 (conf, &st) != -1 && S_ISREG (st.st_mode)))
                found |= read_conf_file (conf, dir, dir_len);
 
              free (conf);