Merge pull request #1200 from janko-m/use-buffer-string-when-parsing-…

…multipart-requests

Use buffer string when parsing multipart requests

lib/rack/multipart/parser.rb

@@ -20,15 +20,15 @@ def initialize(io, content_length)
           @cursor = 0
         end
 
-        def read(size)
+        def read(size, outbuf = nil)
           return if @cursor >= @content_length
 
           left = @content_length - @cursor
 
           str = if left < size
-                  @io.read left
+                  @io.read left, outbuf
                 else
-                  @io.read size
+                  @io.read size, outbuf
                 end
 
           if str
@@ -63,13 +63,14 @@ def self.parse(io, content_length, content_type, tmpfile, bufsize, qp)
         return EMPTY unless boundary
 
         io = BoundedIO.new(io, content_length) if content_length
+        outbuf = String.new
 
         parser = new(boundary, tmpfile, bufsize, qp)
-        parser.on_read io.read(bufsize)
+        parser.on_read io.read(bufsize, outbuf)
 
         loop do
           break if parser.state == :DONE
-          parser.on_read io.read(bufsize)
+          parser.on_read io.read(bufsize, outbuf)
         end
 
         io.rewind