[vlc-devel] [PATCH 3/7] Revert "mkv/demux: Ebml_Parser: fix #18001 (memory-leak)"

Steve Lhomme robux4 at videolabs.io
Mon Nov 13 18:08:42 CET 2017


This reverts commit 37fdb03acd3ec85f73edf6741c97b1a05dc4e197.

This is not the right level to decide if you can free the previous element, you
need at least to respect the "Keep" status.
---
 modules/demux/mkv/Ebml_parser.cpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/modules/demux/mkv/Ebml_parser.cpp b/modules/demux/mkv/Ebml_parser.cpp
index 1e3099cfac..489ff67355 100644
--- a/modules/demux/mkv/Ebml_parser.cpp
+++ b/modules/demux/mkv/Ebml_parser.cpp
@@ -250,8 +250,6 @@ EbmlElement *EbmlParser::Get( int n_call )
             }
 
             delete m_el[mi_level];
-            delete p_prev;
-
             m_el[mi_level] = NULL;
             m_el[mi_level - 1]->SkipData( *m_es, EBML_CONTEXT(m_el[mi_level - 1]) );
             return Get();
-- 
2.14.2



More information about the vlc-devel mailing list