[vlc-commits] [Git][videolan/vlc][master] demux: mkv: fix overflow check

[Hugo Beauzée-Luyssen (@chouquette)]

Hugo Beauzée-Luyssen pushed to branch master at VideoLAN / VLC


Commits:
30430312 by Steve Lhomme at 2022-05-24T14:03:05+00:00
demux: mkv: fix overflow check

It returns true when it overflows.

- - - - -


1 changed file:

- modules/demux/mkv/matroska_segment_parse.cpp


Changes:

=====================================
modules/demux/mkv/matroska_segment_parse.cpp
=====================================
@@ -520,7 +520,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
             }
 
             unsigned int h_crop;
-            if (!add_overflow(i_crop_left, i_crop_right, &h_crop))
+            if (add_overflow(i_crop_left, i_crop_right, &h_crop))
             {
                 debug( vars, "invalid horizontal crop %u+%u",
                               i_crop_left, i_crop_right );
@@ -535,7 +535,7 @@ void matroska_segment_c::ParseTrackEntry( const KaxTrackEntry *m )
             }
 
             unsigned int v_crop;
-            if (!add_overflow(i_crop_top, i_crop_bottom, &v_crop))
+            if (add_overflow(i_crop_top, i_crop_bottom, &v_crop))
             {
                 debug( vars, "invalid vertical crop %u+%u",
                               i_crop_top, i_crop_bottom);



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

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


VideoLAN code repository instance