From 0e6b3a0ba067ca5cbdc766f41c549c20bad9385b Mon Sep 17 00:00:00 2001
From: Dominik Loidolt <dominik.loidolt@univie.ac.at>
Date: Wed, 6 Nov 2024 11:03:29 +0100
Subject: [PATCH] Downgrade reserved field check from error to warning

Make header validation more lenient by not failing decompression when
reserved field is non-zero. Print warning instead of error since this
may indicate corruption but shouldn't prevent decompression attempt.
---
 lib/decompress/decmp.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/lib/decompress/decmp.c b/lib/decompress/decmp.c
index 92ec926..02a826d 100644
--- a/lib/decompress/decmp.c
+++ b/lib/decompress/decmp.c
@@ -1693,10 +1693,8 @@ static int cmp_ent_read_header(const struct cmp_entity *ent, struct cmp_cfg *cfg
 
 	cfg->src = cmp_ent_get_data_buf_const(ent);
 
-	if (cmp_ent_get_reserved(ent)) {
-		debug_print("Error: The reserved field in the compressed header should be zero. Compressed data may be corrupted.");
-		return -1;
-	}
+	if (cmp_ent_get_reserved(ent))
+		debug_print("Warning: The reserved field in the compressed header should be zero.");
 
 	if (cfg->cmp_mode == CMP_MODE_RAW) {
 		if (cmp_ent_get_original_size(ent) != cmp_ent_get_cmp_data_size(ent)) {
-- 
GitLab