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

[dak/master] actually check the type of basedict before trying to use it



Signed-off-by: Mark Hymers <mhy@debian.org>
---
 ChangeLog       |    3 +++
 daklib/utils.py |    6 ++++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 3e3f33e..2991c5b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2008-08-15  Mark Hymers  <mhy@debian.org>
 
+	* daklib/utils.py: Actually check we have basedict before trying to
+	use it.
+
 	*  dak/process_accepted.py, dak/process_unchecked.py,
 	daklib/database.py: Don't change get_files_id to use sha1sum and
 	sha256sum.
diff --git a/daklib/utils.py b/daklib/utils.py
index 34154ce..9ad528a 100755
--- a/daklib/utils.py
+++ b/daklib/utils.py
@@ -220,7 +220,8 @@ def create_hash (lfiles, key, testfn, basedict = None):
             rejmsg.append("Could not open file %s for checksumming" % (f))
 
         # Check hash
-        basedict[f]['%ssum' % key] = testfn(file_handle)
+        if basedict:
+            basedict[f]['%ssum' % key] = testfn(file_handle)
         file_handle.close()
 
     return rejmsg
@@ -248,7 +249,8 @@ def check_hash (where, lfiles, key, testfn, basedict = None):
             rejmsg.append("%s: %s check failed." % (f, key))
         file_handle.close()
         # Store the hashes for later use
-        basedict[f]['%ssum' % key] = lfiles[f][key]
+        if basedict:
+            basedict[f]['%ssum' % key] = lfiles[f][key]
         # Check size
         actual_size = os.stat(f)[stat.ST_SIZE]
         size = int(lfiles[f]["size"])
-- 
1.5.6.3


Reply to: