[vlc-commits] [Git][videolan/vlc][3.0.x] demux: mp4: only return true in GetMoofTrackDuration with a duration

[Steve Lhomme (@robUx4)]

Steve Lhomme pushed to branch 3.0.x at VideoLAN / VLC


Commits:
b6be9a81 by Steve Lhomme at 2025-05-22T06:18:23+00:00
demux: mp4: only return true in GetMoofTrackDuration with a duration

If for some reason there is no traf, we returned true even though p_duration was never set.

(cherry picked from commit b76b5e73257f92156da825855ac6700a3c22ded2)

- - - - -


1 changed file:

- modules/demux/mp4/mp4.c


Changes:

=====================================
modules/demux/mp4/mp4.c
=====================================
@@ -4244,10 +4244,10 @@ static bool GetMoofTrackDuration( MP4_Box_t *p_moov, MP4_Box_t *p_moof,
         }
 
         *p_duration = i_traf_duration;
-        break;
+        return true;
     }
 
-    return true;
+    return false;
 }
 
 static int ProbeFragments( demux_t *p_demux, bool b_force, bool *pb_fragmented )



View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/b6be9a811b0144fbfa8c3b3eb159f74b8b3e2873

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/b6be9a811b0144fbfa8c3b3eb159f74b8b3e2873
You're receiving this email because of your account on code.videolan.org.


VideoLAN code repository instance