simplify safety check logic

The file size would have already been checked by the attrsz check, so this check works implicitly, and works when the file size is not available (streams) Signed-off-by: Kimball Thurston <[email protected]>
AcademySoftwareFoundation · Sep 26, 2024 · 7b3c965 · 7b3c965
1 parent f18a951
commit 7b3c965
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/lib/OpenEXRCore/parse_header.c b/src/lib/OpenEXRCore/parse_header.c
@@ -607,8 +607,7 @@ extract_attr_string_vector (
 
         pulled += sizeof (int32_t);
         nlen = (int32_t) one_to_native32 ((uint32_t) nlen);
-        if (nlen < 0 || nlen > (attrsz - pulled) ||
-            (ctxt->file_size > 0 && nlen > ctxt->file_size))
+        if (nlen < 0 || nlen > (attrsz - pulled))
         {
             rv = ctxt->print_error (
                 ctxt,