commit 6cb36530deafc731cac277080a319d52e4233867
author Dieter Baron <dillo@nih.at> Tue Sep 28 12:07:03 2021 +0200
committer Dieter Baron <dillo@nih.at> Tue Sep 28 12:07:03 2021 +0200
tree d046dd6b3b8093d0e72b20053e30ff302d696c44
parent e7c81b67ab91d5dc54c2238d9f7d9abab1a0a8c3

According to the spec, WinZIP V2 doesn't set CRC.

Closes #261.

lib/zip_dirent.c

diff --git a/lib/zip_dirent.c b/lib/zip_dirent.c
index a7c336b..0fb627e 100644
--- a/lib/zip_dirent.c
+++ b/lib/zip_dirent.c
@@ -664,19 +664,18 @@
 
     crc_valid = true;
     switch (_zip_buffer_get_16(buffer)) {
-    case 1:
-        break;
+        case 1:
+            break;
 
-    case 2:
-        if (de->uncomp_size < 20 /* TODO: constant */) {
+        case 2:
             crc_valid = false;
-        }
-        break;
-
-    default:
-        zip_error_set(error, ZIP_ER_ENCRNOTSUPP, 0);
-        _zip_buffer_free(buffer);
-        return false;
+            /* TODO: When checking consistency, check that crc is 0. */
+            break;
+            
+        default:
+            zip_error_set(error, ZIP_ER_ENCRNOTSUPP, 0);
+            _zip_buffer_free(buffer);
+            return false;
     }
 
     /* vendor */