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

Re: bug: "fatal: cannot pread pack file", version 1.7.5.4

From
Andreas Schwab <schwab@linux-m68k.org>
Date
Jul 2, 2012, 22:43 UTC
Message-ID
<m2vci5n5r5.fsf@igel.home>
In-Reply-To
<20120702215736.GA5790@sigill.intra.peff.net>
Jeff King <peff@peff.net> writes:
> It's very odd for pread to report ENOENT (since it is always operating
> on an already-opened file descriptor).
It doesn't, but gettext will clobber errno:
		n = pread(pack_fd, inbuf, n, from);
		if (n < 0)
			die_errno(_("cannot pread pack file"));

There is nothing that saves errno. This isn't limited to i18n though, any function call in the arguments may potentially clobber errno.

Andreas.
-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."
Previous: Jeff KingNext: Junio C Hamano
Message 4 of 15 in “bug: "fatal: cannot pread pack file", version 1.7.5.4”
  1. Lars WinterfeldJul 2, 2012
  2. Junio C HamanoJul 2, 2012
  3. Jeff KingJul 2, 2012
  4. Andreas SchwabJul 2, 2012
  5. Junio C HamanoJul 2, 2012
  6. Jeff KingJul 3, 2012
  7. Nguyen Thai Ngoc DuyJul 3, 2012
  8. Jeff KingJul 3, 2012
  9. Junio C HamanoJul 3, 2012
  10. Junio C HamanoAug 16, 2012
  11. Lars WinterfeldAug 16, 2012
  12. Junio C HamanoAug 16, 2012
  13. Lars WinterfeldAug 16, 2012
  14. Jeff KingAug 17, 2012
  15. Junio C HamanoAug 17, 2012

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.