* lib/webrick/httpresponse.rb (WEBrick::HTTPResponse#send_body_io):

ensure to close @body. (http://bugs.debian.org/277520)


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@7094 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

ChangeLog

@@ -1,3 +1,8 @@
+Thu Oct 21 19:06:15 2004  GOTOU Yuuzou  <gotoyuzo@notwork.org>
+
+	* lib/webrick/httpresponse.rb (WEBrick::HTTPResponse#send_body_io):
+	  ensure to close @body. (http://bugs.debian.org/277520)
+
 Thu Oct 21 13:11:31 2004  WATANABE Hirofumi  <eban@ruby-lang.org>
 
 	* io.c (pipe_open): variable name "fpw" is conflicted.

lib/webrick/httpresponse.rb

@@ -254,24 +254,27 @@ module WEBrick
     private
 
     def send_body_io(socket)
-      if @request_method == "HEAD"
-        # do nothing
-      elsif chunked?
-        while buf = @body.read(BUFSIZE)
-          next if buf.empty?
-          data = ""
-          data << format("%x", buf.size) << CRLF
-          data << buf << CRLF
-          _write_data(socket, data)
-          @sent_size += buf.size
+      begin
+        if @request_method == "HEAD"
+          # do nothing
+        elsif chunked?
+          while buf = @body.read(BUFSIZE)
+            next if buf.empty?
+            data = ""
+            data << format("%x", buf.size) << CRLF
+            data << buf << CRLF
+            _write_data(socket, data)
+            @sent_size += buf.size
+          end
+          _write_data(socket, "0#{CRLF}#{CRLF}")
+        else
+          size = @header['content-length'].to_i
+          _send_file(socket, @body, 0, size)
+          @sent_size = size
         end
-        _write_data(socket, "0#{CRLF}#{CRLF}")
-      else
-        size = @header['content-length'].to_i
-        _send_file(socket, @body, 0, size)
-        @sent_size = size
+      ensure
+        @body.close
       end
-      @body.close
     end
 
     def send_body_string(socket)