[Apt-listbugs-commits] [SCM] dpkg-ruby development branch, master, updated. debian/0.3.6-2-gec4c124

[Ryan Niebur]

The following commit has been merged in the master branch:
commit ec4c12466540479909ca9dc1b03a83881b824458
Author: Ryan Niebur <ryan at debian.org>
Date:   Sun Sep 4 11:41:40 2011 -0700

    close files once they're parsed (Closes: #585448)

diff --git a/debian/changelog b/debian/changelog
index efcbb55..61ddc05 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,8 +1,12 @@
 dpkg-ruby (0.3.7) UNRELEASED; urgency=low
 
+  [ Francesco Poli (t1000) ]
   * add file name to broken archive exception (Closes: #590146)
 
- -- Francesco Poli (t1000) <frx at firenze.linux.it>  Sun, 25 Jul 2010 16:21:01 +0200
+  [ Ryan Niebur ]
+  * close files once they're parsed (Closes: #585448)
+
+ -- Ryan Niebur <ryan at debian.org>  Sun, 04 Sep 2011 11:41:29 -0700
 
 dpkg-ruby (0.3.6) unstable; urgency=low
 
diff --git a/lib/debian.rb b/lib/debian.rb
index 5bad746..128e5c1 100644
--- a/lib/debian.rb
+++ b/lib/debian.rb
@@ -115,7 +115,10 @@ module Debian
 
     def deb?(debfile)
       begin
-	Debian::Ar.new(debfile).open("debian-binary").read == DEBFORMAT_VERSION
+        f = Debian::Ar.new(debfile)
+        res = (f.open("debian-binary").read == DEBFORMAT_VERSION)
+        f.close
+        return res
       rescue NameError, Debian::ArError
 	false
       end
@@ -568,6 +571,7 @@ module Debian
 	  @data.push(line.chomp)
 	}
       }
+      @artab.close
       freeze
     end
     def control= (c); @control = c; end

-- 
dpkg-ruby development