git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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

From
Johannes Sixt <j.sixt@eudaptics.com>
Date
Jun 13, 2007, 07:05 UTC
Message-ID
<466F971F.C74761E2@eudaptics.com>
In-Reply-To
<20070612213129.GD2459@steel.home>
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
Previous: Alex RiesenNext: Brandon Casey
Message 5 of 11 in “[BUG] 75d8ff1 fails on SunOS 5.9 (missing hsterror) from ba505322”
  1. Randal L. SchwartzJun 12, 2007
  2. Alex RiesenJun 12, 2007
  3. Do not use h_errno after connect(2): the function does not set itAlex Riesen, Jun 12, 2007
  4. Add a local implementation of hstrerror for the system which do not have itAlex Riesen, Jun 12, 2007
  5. Johannes SixtJun 13, 2007
  6. Brandon CaseyJun 18, 2007
  7. Brandon CaseyJun 18, 2007
  8. Alex RiesenJun 18, 2007
  9. Brandon CaseyJun 18, 2007
  10. Add a local implementation of hstrerror for the system which do not have itAlex Riesen, Jun 13, 2007
  11. Randal L. SchwartzJun 16, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.