Skip the rest of userdata box if we encounter an error while parsing it.

This adjusts the behaviour when parsing invalid data in the userdata
box. Prior to this change we'd fail as the parser would be left in a
state we consided invalid. By skipping the rest of the box we instead
continue to parse, and just ignore the userdata.

Author: SingingTree

mp4parse/src/lib.rs

@@ -2344,6 +2344,11 @@ fn read_moov<T: Read>(f: &mut BMFFBox<T>, context: Option<MediaContext>) -> Resu
             BoxType::UserdataBox => {
                 userdata = Some(read_udta(&mut b));
                 debug!("{:?}", userdata);
+                if let Some(Err(_)) = userdata {
+                    // There was an error parsing userdata. Such failures are not fatal to overall
+                    // parsing, just skip the rest of the box.
+                    skip_box_remain(&mut b)?;
+                }
             }
             _ => skip_box_content(&mut b)?,
         };