Subject: Re: [PATCH] Add a local implementation of hstrerror for the system which do not have it

From: Johannes Sixt <[email-masked]>
Date: Wed, 13 Jun 2007 09:05:03 +0200

Alex Riesen wrote:
> +       static char buffer[32];
> ...
> +       sprintf(buffer, "Name resolution error %d", err);

I don't think it's a problem for any current implementation, but it
would be better to err on the conservative side: The buffer should be at
least 44 chars wide to account for 20 digit negative numbers (thats the
maximum if int is 64 bits wide).

-- Hannes