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

[dak/master] parse_file_list: really handle files without an entry in the Files field



We have to raise an exception right away after all if "entry" is None as we
later assign to "entry['sha*sum']".
---
 daklib/upload.py |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/daklib/upload.py b/daklib/upload.py
index 059cc51..1683b8b 100644
--- a/daklib/upload.py
+++ b/daklib/upload.py
@@ -191,6 +191,8 @@ def parse_file_list(control, has_priority_and_section):
             continue
         (sha1sum, size, filename) = line.split()
         entry = entries.get(filename, None)
+        if entry is None:
+            raise InvalidChangesException('{0} is listed in Checksums-Sha1, but not in Files.'.format(filename))
         if entry is not None and entry.get('size', None) != long(size):
             raise InvalidChangesException('Size for {0} in Files and Checksum-Sha1 fields differ.'.format(filename))
         entry['sha1sum'] = sha1sum
@@ -200,6 +202,8 @@ def parse_file_list(control, has_priority_and_section):
             continue
         (sha256sum, size, filename) = line.split()
         entry = entries.get(filename, None)
+        if entry is None:
+            raise InvalidChangesException('{0} is listed in Checksums-Sha256, but not in Files.'.format(filename))
         if entry is not None and entry.get('size', None) != long(size):
             raise InvalidChangesException('Size for {0} in Files and Checksum-Sha256 fields differ.'.format(filename))
         entry['sha256sum'] = sha256sum
-- 
1.7.2.5


Reply to: