]> git.ipfire.org Git - thirdparty/glibc.git/blobdiff - elf/dl-sysdep.c
iconv, localedef: avoid floating point rounding differences [BZ #24372]
[thirdparty/glibc.git] / elf / dl-sysdep.c
index 1ff7a350be5137b800b448abc1c05808e56e7294..5d19b100b2fbd9adcfc0cb7342799778660846fd 100644 (file)
@@ -1,5 +1,5 @@
 /* Operating system support for run-time dynamic linker.  Generic Unix version.
-   Copyright (C) 1995-1998, 2000-2007, 2008 Free Software Foundation, Inc.
+   Copyright (C) 1995-2019 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
 
    The GNU C Library is free software; you can redistribute it and/or
    Lesser General Public License for more details.
 
    You should have received a copy of the GNU Lesser General Public
-   License along with the GNU C Library; if not, write to the Free
-   Software Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
-   02111-1307 USA.  */
+   License along with the GNU C Library; if not, see
+   <http://www.gnu.org/licenses/>.  */
+
+/* We conditionalize the whole of this file rather than simply eliding it
+   from the static build, because other sysdeps/ versions of this file
+   might define things needed by a static build.  */
+
+#ifdef SHARED
 
 #include <assert.h>
 #include <elf.h>
 #include <sys/stat.h>
 #include <sys/mman.h>
 #include <ldsodefs.h>
-#include <stdio-common/_itoa.h>
+#include <_itoa.h>
 #include <fpu_control.h>
 
 #include <entry.h>
 #include <dl-machine.h>
 #include <dl-procinfo.h>
 #include <dl-osinfo.h>
-#include <hp-timing.h>
+#include <libc-internal.h>
 #include <tls.h>
 
-#ifdef _DL_FIRST_PLATFORM
-# define _DL_FIRST_EXTRA (_DL_FIRST_PLATFORM + _DL_PLATFORMS_COUNT)
-#else
-# define _DL_FIRST_EXTRA _DL_HWCAP_COUNT
-#endif
+#include <dl-tunables.h>
 
 extern char **_environ attribute_hidden;
-extern void _end attribute_hidden;
+extern char _end[] attribute_hidden;
 
 /* Protect SUID program against misuse of file descriptors.  */
 extern void __libc_check_standard_fds (void);
@@ -55,13 +56,13 @@ extern void __libc_check_standard_fds (void);
 ElfW(Addr) _dl_base_addr;
 #endif
 int __libc_enable_secure attribute_relro = 0;
-INTVARDEF(__libc_enable_secure)
+rtld_hidden_data_def (__libc_enable_secure)
 int __libc_multiple_libcs = 0; /* Defining this here avoids the inclusion
                                   of init-first.  */
 /* This variable contains the lowest stack address ever used.  */
 void *__libc_stack_end attribute_relro = NULL;
 rtld_hidden_data_def(__libc_stack_end)
-static ElfW(auxv_t) *_dl_auxv attribute_relro;
+void *_dl_random attribute_relro = NULL;
 
 #ifndef DL_FIND_ARG_COMPONENTS
 # define DL_FIND_ARG_COMPONENTS(cookie, argc, argv, envp, auxp)        \
@@ -83,7 +84,7 @@ static ElfW(auxv_t) *_dl_auxv attribute_relro;
 ElfW(Addr)
 _dl_sysdep_start (void **start_argptr,
                  void (*dl_main) (const ElfW(Phdr) *phdr, ElfW(Word) phnum,
-                                  ElfW(Addr) *user_entry))
+                                  ElfW(Addr) *user_entry, ElfW(auxv_t) *auxv))
 {
   const ElfW(Phdr) *phdr = NULL;
   ElfW(Word) phnum = 0;
@@ -109,13 +110,13 @@ _dl_sysdep_start (void **start_argptr,
 #endif
 
   __libc_stack_end = DL_STACK_END (start_argptr);
-  DL_FIND_ARG_COMPONENTS (start_argptr, _dl_argc, INTUSE(_dl_argv), _environ,
-                         _dl_auxv);
+  DL_FIND_ARG_COMPONENTS (start_argptr, _dl_argc, _dl_argv, _environ,
+                         GLRO(dl_auxv));
 
   user_entry = (ElfW(Addr)) ENTRY_POINT;
   GLRO(dl_platform) = NULL; /* Default to nothing known about the platform.  */
 
-  for (av = _dl_auxv; av->a_type != AT_NULL; set_seen (av++))
+  for (av = GLRO(dl_auxv); av->a_type != AT_NULL; set_seen (av++))
     switch (av->a_type)
       {
       case AT_PHDR:
@@ -149,7 +150,7 @@ _dl_sysdep_start (void **start_argptr,
 #ifndef HAVE_AUX_SECURE
        seen = -1;
 #endif
-       INTUSE(__libc_enable_secure) = av->a_un.a_val;
+       __libc_enable_secure = av->a_un.a_val;
        break;
       case AT_PLATFORM:
        GLRO(dl_platform) = (void *) av->a_un.a_val;
@@ -157,6 +158,9 @@ _dl_sysdep_start (void **start_argptr,
       case AT_HWCAP:
        GLRO(dl_hwcap) = (unsigned long int) av->a_un.a_val;
        break;
+      case AT_HWCAP2:
+       GLRO(dl_hwcap2) = (unsigned long int) av->a_un.a_val;
+       break;
       case AT_CLKTCK:
        GLRO(dl_clktck) = av->a_un.a_val;
        break;
@@ -168,13 +172,13 @@ _dl_sysdep_start (void **start_argptr,
        new_sysinfo = av->a_un.a_val;
        break;
 #endif
-#if defined NEED_DL_SYSINFO || defined NEED_DL_SYSINFO_DSO
+#ifdef NEED_DL_SYSINFO_DSO
       case AT_SYSINFO_EHDR:
        GLRO(dl_sysinfo_dso) = (void *) av->a_un.a_val;
        break;
 #endif
-      case AT_EXECFN:
-       GLRO(dl_execfn) = (void *) av->a_un.a_val;
+      case AT_RANDOM:
+       _dl_random = (void *) av->a_un.a_val;
        break;
 #ifdef DL_PLATFORM_AUXV
       DL_PLATFORM_AUXV
@@ -197,7 +201,7 @@ _dl_sysdep_start (void **start_argptr,
 
       /* If one of the two pairs of IDs does not match this is a setuid
         or setgid run.  */
-      INTUSE(__libc_enable_secure) = uid | gid;
+      __libc_enable_secure = uid | gid;
     }
 #endif
 
@@ -206,12 +210,19 @@ _dl_sysdep_start (void **start_argptr,
     GLRO(dl_pagesize) = __getpagesize ();
 #endif
 
-#if defined NEED_DL_SYSINFO
-  /* Only set the sysinfo value if we also have the vsyscall DSO.  */
-  if (GLRO(dl_sysinfo_dso) != 0 && new_sysinfo)
-    GLRO(dl_sysinfo) = new_sysinfo;
+#ifdef NEED_DL_SYSINFO
+  if (new_sysinfo != 0)
+    {
+# ifdef NEED_DL_SYSINFO_DSO
+      /* Only set the sysinfo value if we also have the vsyscall DSO.  */
+      if (GLRO(dl_sysinfo_dso) != 0)
+# endif
+        GLRO(dl_sysinfo) = new_sysinfo;
+    }
 #endif
 
+  __tunables_init (_environ);
+
 #ifdef DL_SYSDEP_INIT
   DL_SYSDEP_INIT;
 #endif
@@ -224,33 +235,31 @@ _dl_sysdep_start (void **start_argptr,
   if (GLRO(dl_platform) != NULL)
     GLRO(dl_platformlen) = strlen (GLRO(dl_platform));
 
-  if (__sbrk (0) == &_end)
+  if (__sbrk (0) == _end)
     /* The dynamic linker was run as a program, and so the initial break
        starts just after our bss, at &_end.  The malloc in dl-minimal.c
        will consume the rest of this page, so tell the kernel to move the
        break up that far.  When the user program examines its break, it
        will see this new value and not clobber our data.  */
     __sbrk (GLRO(dl_pagesize)
-           - ((&_end - (void *) 0) & (GLRO(dl_pagesize) - 1)));
+           - ((_end - (char *) 0) & (GLRO(dl_pagesize) - 1)));
 
   /* If this is a SUID program we make sure that FDs 0, 1, and 2 are
      allocated.  If necessary we are doing it ourself.  If it is not
      possible we stop the program.  */
-  if (__builtin_expect (INTUSE(__libc_enable_secure), 0))
+  if (__builtin_expect (__libc_enable_secure, 0))
     __libc_check_standard_fds ();
 
-  (*dl_main) (phdr, phnum, &user_entry);
+  (*dl_main) (phdr, phnum, &user_entry, GLRO(dl_auxv));
   return user_entry;
 }
 
 void
-internal_function
 _dl_sysdep_start_cleanup (void)
 {
 }
 
 void
-internal_function
 _dl_show_auxv (void)
 {
   char buf[64];
@@ -264,53 +273,64 @@ _dl_show_auxv (void)
      close by (otherwise the array will be too large).  In case we have
      to support a platform where these requirements are not fulfilled
      some alternative implementation has to be used.  */
-  for (av = _dl_auxv; av->a_type != AT_NULL; ++av)
+  for (av = GLRO(dl_auxv); av->a_type != AT_NULL; ++av)
     {
       static const struct
       {
-       const char label[20];
-       enum { unknown = 0, dec, hex, str, ignore } form;
+       const char label[22];
+       enum { unknown = 0, dec, hex, str, ignore } form : 8;
       } auxvars[] =
        {
-         [AT_EXECFD - 2] =             { "AT_EXECFD:       ", dec },
-         [AT_PHDR - 2] =               { "AT_PHDR:         0x", hex },
-         [AT_PHENT - 2] =              { "AT_PHENT:        ", dec },
-         [AT_PHNUM - 2] =              { "AT_PHNUM:        ", dec },
-         [AT_PAGESZ - 2] =             { "AT_PAGESZ:       ", dec },
-         [AT_BASE - 2] =               { "AT_BASE:         0x", hex },
-         [AT_FLAGS - 2] =              { "AT_FLAGS:        0x", hex },
-         [AT_ENTRY - 2] =              { "AT_ENTRY:        0x", hex },
-         [AT_NOTELF - 2] =             { "AT_NOTELF:       ", hex },
-         [AT_UID - 2] =                { "AT_UID:          ", dec },
-         [AT_EUID - 2] =               { "AT_EUID:         ", dec },
-         [AT_GID - 2] =                { "AT_GID:          ", dec },
-         [AT_EGID - 2] =               { "AT_EGID:         ", dec },
-         [AT_PLATFORM - 2] =           { "AT_PLATFORM:     ", str },
-         [AT_HWCAP - 2] =              { "AT_HWCAP:        ", hex },
-         [AT_CLKTCK - 2] =             { "AT_CLKTCK:       ", dec },
-         [AT_FPUCW - 2] =              { "AT_FPUCW:        ", hex },
-         [AT_DCACHEBSIZE - 2] =        { "AT_DCACHEBSIZE:  0x", hex },
-         [AT_ICACHEBSIZE - 2] =        { "AT_ICACHEBSIZE:  0x", hex },
-         [AT_UCACHEBSIZE - 2] =        { "AT_UCACHEBSIZE:  0x", hex },
-         [AT_IGNOREPPC - 2] =          { "AT_IGNOREPPC", ignore },
-         [AT_SECURE - 2] =             { "AT_SECURE:       ", dec },
-         [AT_SYSINFO - 2] =            { "AT_SYSINFO:      0x", hex },
-         [AT_SYSINFO_EHDR - 2] =       { "AT_SYSINFO_EHDR: 0x", hex },
+         [AT_EXECFD - 2] =             { "EXECFD:            ", dec },
+         [AT_EXECFN - 2] =             { "EXECFN:            ", str },
+         [AT_PHDR - 2] =               { "PHDR:              0x", hex },
+         [AT_PHENT - 2] =              { "PHENT:             ", dec },
+         [AT_PHNUM - 2] =              { "PHNUM:             ", dec },
+         [AT_PAGESZ - 2] =             { "PAGESZ:            ", dec },
+         [AT_BASE - 2] =               { "BASE:              0x", hex },
+         [AT_FLAGS - 2] =              { "FLAGS:             0x", hex },
+         [AT_ENTRY - 2] =              { "ENTRY:             0x", hex },
+         [AT_NOTELF - 2] =             { "NOTELF:            ", hex },
+         [AT_UID - 2] =                { "UID:               ", dec },
+         [AT_EUID - 2] =               { "EUID:              ", dec },
+         [AT_GID - 2] =                { "GID:               ", dec },
+         [AT_EGID - 2] =               { "EGID:              ", dec },
+         [AT_PLATFORM - 2] =           { "PLATFORM:          ", str },
+         [AT_HWCAP - 2] =              { "HWCAP:             ", hex },
+         [AT_CLKTCK - 2] =             { "CLKTCK:            ", dec },
+         [AT_FPUCW - 2] =              { "FPUCW:             ", hex },
+         [AT_DCACHEBSIZE - 2] =        { "DCACHEBSIZE:       0x", hex },
+         [AT_ICACHEBSIZE - 2] =        { "ICACHEBSIZE:       0x", hex },
+         [AT_UCACHEBSIZE - 2] =        { "UCACHEBSIZE:       0x", hex },
+         [AT_IGNOREPPC - 2] =          { "IGNOREPPC", ignore },
+         [AT_SECURE - 2] =             { "SECURE:            ", dec },
+         [AT_BASE_PLATFORM - 2] =      { "BASE_PLATFORM:     ", str },
+         [AT_SYSINFO - 2] =            { "SYSINFO:           0x", hex },
+         [AT_SYSINFO_EHDR - 2] =       { "SYSINFO_EHDR:      0x", hex },
+         [AT_RANDOM - 2] =             { "RANDOM:            0x", hex },
+         [AT_HWCAP2 - 2] =             { "HWCAP2:            0x", hex },
+         [AT_L1I_CACHESIZE - 2] =      { "L1I_CACHESIZE:     ", dec },
+         [AT_L1I_CACHEGEOMETRY - 2] =  { "L1I_CACHEGEOMETRY: 0x", hex },
+         [AT_L1D_CACHESIZE - 2] =      { "L1D_CACHESIZE:     ", dec },
+         [AT_L1D_CACHEGEOMETRY - 2] =  { "L1D_CACHEGEOMETRY: 0x", hex },
+         [AT_L2_CACHESIZE - 2] =       { "L2_CACHESIZE:      ", dec },
+         [AT_L2_CACHEGEOMETRY - 2] =   { "L2_CACHEGEOMETRY:  0x", hex },
+         [AT_L3_CACHESIZE - 2] =       { "L3_CACHESIZE:      ", dec },
+         [AT_L3_CACHEGEOMETRY - 2] =   { "L3_CACHEGEOMETRY:  0x", hex },
        };
       unsigned int idx = (unsigned int) (av->a_type - 2);
 
-      if ((unsigned int) av->a_type < 2u || auxvars[idx].form == ignore)
+      if ((unsigned int) av->a_type < 2u
+         || (idx < sizeof (auxvars) / sizeof (auxvars[0])
+             && auxvars[idx].form == ignore))
        continue;
 
       assert (AT_NULL == 0);
       assert (AT_IGNORE == 1);
 
-      if (av->a_type == AT_HWCAP)
-       {
-         /* This is handled special.  */
-         if (_dl_procinfo (av->a_un.a_val) == 0)
-           continue;
-       }
+      /* Some entries are handled in a special way per platform.  */
+      if (_dl_procinfo (av->a_type, av->a_un.a_val) == 0)
+       continue;
 
       if (idx < sizeof (auxvars) / sizeof (auxvars[0])
          && auxvars[idx].form != unknown)
@@ -324,14 +344,14 @@ _dl_show_auxv (void)
            val = _itoa ((unsigned long int) av->a_un.a_val,
                         buf + sizeof buf - 1, 16, 0);
 
-         _dl_printf ("%s%s\n", auxvars[idx].label, val);
+         _dl_printf ("AT_%s%s\n", auxvars[idx].label, val);
 
          continue;
        }
 
       /* Unknown value: print a generic line.  */
       char buf2[17];
-      buf[sizeof (buf2) - 1] = '\0';
+      buf2[sizeof (buf2) - 1] = '\0';
       const char *val2 = _itoa ((unsigned long int) av->a_un.a_val,
                                buf2 + sizeof buf2 - 1, 16, 0);
       const char *val =  _itoa ((unsigned long int) av->a_type,
@@ -340,245 +360,4 @@ _dl_show_auxv (void)
     }
 }
 
-
-/* Return an array of useful/necessary hardware capability names.  */
-const struct r_strlenpair *
-internal_function
-_dl_important_hwcaps (const char *platform, size_t platform_len, size_t *sz,
-                     size_t *max_capstrlen)
-{
-  /* Determine how many important bits are set.  */
-  uint64_t masked = GLRO(dl_hwcap) & GLRO(dl_hwcap_mask);
-  size_t cnt = platform != NULL;
-  size_t n, m;
-  size_t total;
-  struct r_strlenpair *temp;
-  struct r_strlenpair *result;
-  struct r_strlenpair *rp;
-  char *cp;
-
-  /* Count the number of bits set in the masked value.  */
-  for (n = 0; (~((1ULL << n) - 1) & masked) != 0; ++n)
-    if ((masked & (1ULL << n)) != 0)
-      ++cnt;
-
-#if (defined NEED_DL_SYSINFO || defined NEED_DL_SYSINFO_DSO) && defined SHARED
-  /* The system-supplied DSO can contain a note of type 2, vendor "GNU".
-     This gives us a list of names to treat as fake hwcap bits.  */
-
-  const char *dsocaps = NULL;
-  size_t dsocapslen = 0;
-  if (GLRO(dl_sysinfo_map) != NULL)
-    {
-      const ElfW(Phdr) *const phdr = GLRO(dl_sysinfo_map)->l_phdr;
-      const ElfW(Word) phnum = GLRO(dl_sysinfo_map)->l_phnum;
-      for (uint_fast16_t i = 0; i < phnum; ++i)
-       if (phdr[i].p_type == PT_NOTE)
-         {
-           const ElfW(Addr) start = (phdr[i].p_vaddr
-                                     + GLRO(dl_sysinfo_map)->l_addr);
-           const struct
-           {
-             ElfW(Word) vendorlen;
-             ElfW(Word) datalen;
-             ElfW(Word) type;
-           } *note = (const void *) start;
-           while ((ElfW(Addr)) (note + 1) - start < phdr[i].p_memsz)
-             {
-#define ROUND(len) (((len) + sizeof (ElfW(Word)) - 1) & -sizeof (ElfW(Word)))
-               if (note->type == 2
-                   && note->vendorlen == sizeof "GNU"
-                   && !memcmp ((note + 1), "GNU", sizeof "GNU")
-                   && note->datalen > 2 * sizeof (ElfW(Word)) + 2)
-                 {
-                   const ElfW(Word) *p = ((const void *) (note + 1)
-                                          + ROUND (sizeof "GNU"));
-                   cnt += *p++;
-                   ++p;        /* Skip mask word.  */
-                   dsocaps = (const char *) p;
-                   dsocapslen = note->datalen - sizeof *p * 2;
-                   break;
-                 }
-               note = ((const void *) (note + 1)
-                       + ROUND (note->vendorlen) + ROUND (note->datalen));
-#undef ROUND
-             }
-           if (dsocaps != NULL)
-             break;
-         }
-    }
-#endif
-
-  /* For TLS enabled builds always add 'tls'.  */
-  ++cnt;
-
-  /* Create temporary data structure to generate result table.  */
-  temp = (struct r_strlenpair *) alloca (cnt * sizeof (*temp));
-  m = 0;
-#if defined NEED_DL_SYSINFO || defined NEED_DL_SYSINFO_DSO
-  if (dsocaps != NULL)
-    {
-      const ElfW(Word) mask = ((const ElfW(Word) *) dsocaps)[-1];
-      GLRO(dl_hwcap) |= (uint64_t) mask << _DL_FIRST_EXTRA;
-      size_t len;
-      for (const char *p = dsocaps; p < dsocaps + dsocapslen; p += len + 1)
-       {
-         uint_fast8_t bit = *p++;
-         len = strlen (p);
-
-         /* Skip entries that are not enabled in the mask word.  */
-         if (__builtin_expect (mask & ((ElfW(Word)) 1 << bit), 1))
-           {
-             temp[m].str = p;
-             temp[m].len = len;
-             ++m;
-           }
-         else
-           --cnt;
-       }
-    }
 #endif
-  for (n = 0; masked != 0; ++n)
-    if ((masked & (1ULL << n)) != 0)
-      {
-       temp[m].str = _dl_hwcap_string (n);
-       temp[m].len = strlen (temp[m].str);
-       masked ^= 1ULL << n;
-       ++m;
-      }
-  if (platform != NULL)
-    {
-      temp[m].str = platform;
-      temp[m].len = platform_len;
-      ++m;
-    }
-
-  temp[m].str = "tls";
-  temp[m].len = 3;
-  ++m;
-
-  assert (m == cnt);
-
-  /* Determine the total size of all strings together.  */
-  if (cnt == 1)
-    total = temp[0].len + 1;
-  else
-    {
-      total = temp[0].len + temp[cnt - 1].len + 2;
-      if (cnt > 2)
-       {
-         total <<= 1;
-         for (n = 1; n + 1 < cnt; ++n)
-           total += temp[n].len + 1;
-         if (cnt > 3
-             && (cnt >= sizeof (size_t) * 8
-                 || total + (sizeof (*result) << 3)
-                    >= (1UL << (sizeof (size_t) * 8 - cnt + 3))))
-           _dl_signal_error (ENOMEM, NULL, NULL,
-                             N_("cannot create capability list"));
-
-         total <<= cnt - 3;
-       }
-    }
-
-  /* The result structure: we use a very compressed way to store the
-     various combinations of capability names.  */
-  *sz = 1 << cnt;
-  result = (struct r_strlenpair *) malloc (*sz * sizeof (*result) + total);
-  if (result == NULL)
-    _dl_signal_error (ENOMEM, NULL, NULL,
-                     N_("cannot create capability list"));
-
-  if (cnt == 1)
-    {
-      result[0].str = (char *) (result + *sz);
-      result[0].len = temp[0].len + 1;
-      result[1].str = (char *) (result + *sz);
-      result[1].len = 0;
-      cp = __mempcpy ((char *) (result + *sz), temp[0].str, temp[0].len);
-      *cp = '/';
-      *sz = 2;
-      *max_capstrlen = result[0].len;
-
-      return result;
-    }
-
-  /* Fill in the information.  This follows the following scheme
-     (indeces from TEMP for four strings):
-       entry #0: 0, 1, 2, 3    binary: 1111
-             #1: 0, 1, 3               1101
-             #2: 0, 2, 3               1011
-             #3: 0, 3                  1001
-     This allows the representation of all possible combinations of
-     capability names in the string.  First generate the strings.  */
-  result[1].str = result[0].str = cp = (char *) (result + *sz);
-#define add(idx) \
-      cp = __mempcpy (__mempcpy (cp, temp[idx].str, temp[idx].len), "/", 1);
-  if (cnt == 2)
-    {
-      add (1);
-      add (0);
-    }
-  else
-    {
-      n = 1 << (cnt - 1);
-      do
-       {
-         n -= 2;
-
-         /* We always add the last string.  */
-         add (cnt - 1);
-
-         /* Add the strings which have the bit set in N.  */
-         for (m = cnt - 2; m > 0; --m)
-           if ((n & (1 << m)) != 0)
-             add (m);
-
-         /* Always add the first string.  */
-         add (0);
-       }
-      while (n != 0);
-    }
-#undef add
-
-  /* Now we are ready to install the string pointers and length.  */
-  for (n = 0; n < (1UL << cnt); ++n)
-    result[n].len = 0;
-  n = cnt;
-  do
-    {
-      size_t mask = 1 << --n;
-
-      rp = result;
-      for (m = 1 << cnt; m > 0; ++rp)
-       if ((--m & mask) != 0)
-         rp->len += temp[n].len + 1;
-    }
-  while (n != 0);
-
-  /* The first half of the strings all include the first string.  */
-  n = (1 << cnt) - 2;
-  rp = &result[2];
-  while (n != (1UL << (cnt - 1)))
-    {
-      if ((--n & 1) != 0)
-       rp[0].str = rp[-2].str + rp[-2].len;
-      else
-       rp[0].str = rp[-1].str;
-      ++rp;
-    }
-
-  /* The second half starts right after the first part of the string of
-     the corresponding entry in the first half.  */
-  do
-    {
-      rp[0].str = rp[-(1 << (cnt - 1))].str + temp[cnt - 1].len + 1;
-      ++rp;
-    }
-  while (--n != 0);
-
-  /* The maximum string length.  */
-  *max_capstrlen = result[0].len;
-
-  return result;
-}