about summary refs log tree commit homepage
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2019-05-04 16:13:17 +0000
committerEric Wong <e@80x24.org>2019-05-04 16:18:50 +0000
commitb0f439c2246c265a2b2f3f8cc6afbfb58df95fd9 (patch)
treee1184b5efa4c6cfbe7d92d399f6407d2f79a6231
parent0ebf8a48918b476df9fc7c8b4f8617ab08ca0d16 (diff)
downloadyahns-b0f439c2246c265a2b2f3f8cc6afbfb58df95fd9.tar.gz
Linux before 4.9 (and before 3.16.57) failed to account
for the existing size of a pipe before checking system
resource limits and would return EPERM in that case.

https://80x24.org/mirrors/linux.git/commit?id=b0b91d18e2e97b741b294af9333824ecc3fadfd8
https://lore.kernel.org/lkml/?q=s%3A%22fix+limit+checking+in+pipe_set_size%22

Based on a patch by Stephen Demjanenko for unicorn:
https://bogomips.org/unicorn-public/1556922018-24096-1-git-send-email-sdemjanenko@gmail.com/
-rw-r--r--lib/yahns/worker.rb4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/yahns/worker.rb b/lib/yahns/worker.rb
index 0355629..ec5be23 100644
--- a/lib/yahns/worker.rb
+++ b/lib/yahns/worker.rb
@@ -14,8 +14,8 @@ class Yahns::Worker # :nodoc:
       # F_SETPIPE_SZ = 1031, PAGE_SIZE = 4096
       # (fcntl will handle minimum size on platforms where PAGE_SIZE > 4096)
       @to_io.fcntl(1031, 4096)
-    rescue Errno::EINVAL
-      # old kernel
+    rescue SystemCallError
+      # old kernel (EINVAL, EPERM)
     end if RUBY_PLATFORM =~ /\blinux\b/
   end