Re: [bug-gnulib] iconvme again

[Paul Eggert]

Simon Josefsson <address@hidden> writes:

> Hello.  iconvme.* has been installed into libc.  This patch propose to
> add it to gnulib via libc.  What do you think?

There was something wrong with that email.  It claimed to have a file
iconvme.h, but the contents of that file were identical to the
contents of iconvme.c.  What caused that problem?  Can you please
resubmit the suggestion?

I did notice one unchecked arithmetic overflow.  This:

> +  size_t outbuf_size = (inbytes_remaining + 1) * MB_LEN_MAX;

can overflow and cause buffer overruns.  I suggest adding something
like this as a conservative check, just after the declarations:

    if (1 < MB_LEN_MAX && SIZE_MAX / MB_LEN_MAX <= inbytes_remaining)
      {
        errno = ENOMEM;
        return NULL;
      }