[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

[SCM] Debian package checker branch, master, updated. 2.5.10-193-g2ac1f0d



The following commit has been merged in the master branch:
commit 2ac1f0dc18cc669130f272e307e24d65a446bb5f
Author: Niels Thykier <niels@thykier.net>
Date:   Fri Oct 12 21:25:38 2012 +0200

    harness:  Fix missing checking of return value of close
    
    Fix a missing check of the return value of closing of two write
    handles.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/reporting/harness b/reporting/harness
index 8716fe8..8127fea 100755
--- a/reporting/harness
+++ b/reporting/harness
@@ -302,8 +302,8 @@ unless ($opt{'reports-only'}) {
             }
         }
         print $nfd "N: ---end-of-old-lintian-log-file---\n";
-        close $nfd;
-        close $ofd;
+        close $nfd or Die ("Close $lintian_log: $!");
+        close $ofd; # Ignore (read-only handle)
         Log ('');
         if (@inc) {
             Log ('Creating work list for lintian');
@@ -313,7 +313,7 @@ unless ($opt{'reports-only'}) {
                 foreach my $query (@inc) {
                     print $lfd "!query: $query\n";
                 }
-                close $lfd;
+                close $lfd or Die ("Close $list_file: $!");
             }
             Log ('');
 

-- 
Debian package checker


Reply to: