StringIO#readlines also needs limit=0 check.

jruby · Aug 5, 2017 · e21e4e1 · e21e4e1
1 parent abde7ba
commit e21e4e1
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/core/src/main/java/org/jruby/ext/stringio/StringIO.java b/core/src/main/java/org/jruby/ext/stringio/StringIO.java
@@ -622,6 +622,10 @@ public RubyArray getline(ThreadContext context, StringIO self, IRubyObject rs, i
             RubyArray ary = context.runtime.newArray();
             IRubyObject line;
 
+            if (limit == 0) {
+                throw context.runtime.newArgumentError("invalid limit: 0 for readlines");
+            }
+
             while (!(line = self.getline(context, rs, limit, chomp)).isNil()) {
                 ary.append(line);
             }