]> Sergey Matveev's repositories - public-inbox.git/commitdiff
http: reuse STDIN if it's already /dev/null
authorEric Wong <e@80x24.org>
Tue, 24 Jan 2023 09:49:35 +0000 (09:49 +0000)
committerEric Wong <e@80x24.org>
Tue, 24 Jan 2023 10:07:54 +0000 (10:07 +0000)
It's typical for -netd/-httpd to have STDIN pointed to
/dev/null, so try to use that instead of opening another
file description.

lib/PublicInbox/HTTP.pm

index 0dba425dd6354d4925fa82b49a24ea669a16db76..ca162939c9f1ea47ecefe47333139eff9e42a0ed 100644 (file)
@@ -43,7 +43,13 @@ use Errno qw(EAGAIN);
 our $MAX_REQUEST_BUFFER = $ENV{GIT_HTTP_MAX_REQUEST_BUFFER} ||
                        (10 * 1024 * 1024);
 
-open(my $null_io, '<', '/dev/null') or die "failed to open /dev/null: $!";
+open(my $null_io, '<', '/dev/null') or die "open /dev/null: $!";
+{
+       my @n = stat($null_io) or die "stat(/dev/null): $!";
+       my @i = stat(STDIN) or die "stat(STDIN): $!";
+       $null_io = *STDIN{IO} if "@n[0, 1]" eq "@i[0, 1]";
+}
+
 my $http_date;
 my $prev = 0;
 sub http_date () {