fix crashes reported by fuzz testing - avoid hang while printing

mindeng · Jul 12, 2024 · 8d86166 · 8d86166
1 parent bb8a1f0
commit 8d86166
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/examples/rexiftool.rs b/examples/rexiftool.rs
@@ -172,7 +172,7 @@ fn main() -> Result<(), Box<dyn Error>> {
                 .collect::<Vec<_>>()
         }
         other => {
-            println!("Unsupported filetype: {other}");
+            eprintln!("Unsupported filetype: {other}");
             return Err("Unsupported filetype".into());
         }
     };

diff --git a/src/bbox.rs b/src/bbox.rs
@@ -77,7 +77,20 @@ impl BoxHeader {
         }
 
         if box_size > (MAX_BODY_LEN + header_size) as u64 {
-            eprintln!("box size of box '{}' is too big: {}", box_type, box_size);
+            eprintln!(
+                "box size of box '{}' is too big: {}",
+                box_type
+                    .chars()
+                    .map(|c| {
+                        if c.is_ascii_graphic() {
+                            c.as_char()
+                        } else {
+                            '*'
+                        }
+                    })
+                    .collect::<String>(),
+                box_size
+            );
             return fail(remain);
         }