At line 589 of coreutils/lib/locale_charset.c, var ‘aliases' points to a buffer which is allocated using malloc() .This buffer is not freed when codeset is still an empty string after the loop (Line 589~597).

So it will be leaked under such situation.

Our static analysis tool reports this problem.

   /* Resolve alias. */

   for (aliases = get_charset_aliases ();

        *aliases != '\0';

        aliases += strlen (aliases) + 1, aliases += strlen (aliases) + 1)

     if (strcmp (codeset, aliases) == 0

         || (aliases[0] == '*' && aliases[1] == '\0'))

{

         codeset = aliases + strlen (aliases) + 1;

         break;

}

598

   /* Don't return an empty string.  GNU libc and GNU libiconv interpret

      the empty string as denoting "the locale's character encoding",

      thus GNU libiconv would call this function a second time.  */

   if (codeset[0] == '\0')

     codeset = "ASCII”;

 
 #ifdef DARWIN7
   /* Mac OS X sets MB_CUR_MAX to 1 when LC_ALL=C, and "UTF-8"
      (the default codeset) does not work when MB_CUR_MAX is 1.  */
   if (strcmp (codeset, "UTF-8") == 0 && MB_CUR_MAX_L (uselocale (NULL)) <= 1)
     codeset = "ASCII";
 #endif
 
   return codeset;
 }