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

[PATCH 3/3] gitweb: remove test when closing file descriptor

From
Sylvain Rabot <sylvain@abstraction.fr>
Date
Dec 20, 2010, 19:01 UTC
Message-ID
<1292871681-4818-4-git-send-email-sylvain@abstraction.fr>
In-Reply-To
<1292871681-4818-1-git-send-email-sylvain@abstraction.fr>

it happens that closing file descriptor fails whereas the blob is perfectly readable. According to perlman the reasons could be:

   If the file handle came from a piped open, "close" will additionally
   return false if one of the other system calls involved fails, or if the
   program exits with non-zero status.  (If the only problem was that the
   program exited non-zero, $! will be set to 0.)  Closing a pipe also waits
   for the process executing on the pipe to complete, in case you want to
   look at the output of the pipe afterwards, and implicitly puts the exit
   status value of that command into $?.
   Prematurely closing the read end of a pipe (i.e. before the process writ-
   ing to it at the other end has closed it) will result in a SIGPIPE being
   delivered to the writer.  If the other end can't handle that, be sure to
   read all the data before closing the pipe.
In this case we don't mind that close fails.
Signed-off-by: Sylvain Rabot <sylvain@abstraction.fr>
---
 gitweb/gitweb.perl |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
index c460f27..8f0d00e 100755
--- a/gitweb/gitweb.perl
+++ b/gitweb/gitweb.perl
@@ -3450,8 +3450,7 @@ sub run_highlighter {
 	my ($fd, $highlight, $syntax) = @_;
 	return $fd unless ($highlight && defined $syntax);
 
-	close $fd
-		or die_error(404, "Reading blob failed");
+	close $fd;
 	open $fd, quote_command(git_cmd(), "cat-file", "blob", $hash)." | ".
 	          quote_command($highlight_bin).
 	          " --xhtml --fragment --syntax $syntax |"
-- 
1.7.3.4.523.g72f0d.dirty
Previous: Jakub Narebski
Message 7 of 7 in “minor gitweb modifications”
  1. 0/3 minor gitweb modificationsSylvain Rabot, Dec 20, 2010
  2. 1/3 gitweb: add extensions to highlight featureSylvain Rabot, Dec 20, 2010
  3. 2/3 gitweb: decorate a bit more remotesSylvain Rabot, Dec 20, 2010
  4. Jonathan NiederDec 20, 2010
  5. Jakub NarebskiDec 20, 2010
  6. Jakub NarebskiDec 20, 2010
  7. 3/3 gitweb: remove test when closing file descriptorSylvain Rabot, Dec 20, 2010

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.