[vlc-commits] [Git][videolan/vlc][3.0.x] demux: mkv: allow infinite elements in finite parent

[Steve Lhomme (@robUx4)]

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


Commits:
a438aba4 by Steve Lhomme at 2026-09-30T10:27:21+02:00
demux: mkv: allow infinite elements in finite parent

The stack overflow from #29578 is no longer present in that sample with
all the libebml and recent demuxer changes. We can allow this type of file again.

Fixes #30147
Fixes #30172

(cherry picked from commit fbc9f55971c338d0d06de1d2fc60646839ec698e)
Signed-off-by: Steve Lhomme <robux4 at ycbcr.xyz>

- - - - -


1 changed file:

- modules/demux/mkv/Ebml_parser.cpp


Changes:

=====================================
modules/demux/mkv/Ebml_parser.cpp
=====================================
@@ -160,11 +160,8 @@ EbmlElement *EbmlParser::Get( bool allow_overshoot )
             }
             if( !ret->IsFiniteSize() )
             {
-                msg_Err( p_demux, "Infinite EBML element %s at %" PRIu64 " inside finite parent",
+                msg_Dbg( p_demux, "Infinite EBML element %s at %" PRIu64 " inside finite parent",
                          EBML_NAME(ret), ret->GetElementPosition() );
-                delete ret;
-                m_el[mi_level] = NULL;
-                return NULL;
             }
         }
         return ret;



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

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/commit/a438aba4927380c3593e628618181e0c91012bc6
You're receiving this email because of your account on code.videolan.org. Manage all notifications: https://code.videolan.org/-/profile/notifications | Help: https://code.videolan.org/help