[vlc-commits] demux:mkv: fix double free introduced

Steve Lhomme git at videolan.org
Mon Nov 27 20:58:01 CET 2017


vlc | branch: master | Steve Lhomme <robUx4 at videolabs.io> | Mon Nov 27 17:31:36 2017 +0100| [e77202ea2d4e05b8e3b08ea8cb7a762cd239eae0] | committer: Jean-Baptiste Kempf

demux:mkv: fix double free introduced

Introduced by e527210321013fd477673c36c472e3ca9a2a9fff.
There is a local parser, we don't need to free it. Instead it freed an upper one.

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=e77202ea2d4e05b8e3b08ea8cb7a762cd239eae0
---

 modules/demux/mkv/matroska_segment.cpp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/modules/demux/mkv/matroska_segment.cpp b/modules/demux/mkv/matroska_segment.cpp
index 52be3711c7..bca4f68571 100644
--- a/modules/demux/mkv/matroska_segment.cpp
+++ b/modules/demux/mkv/matroska_segment.cpp
@@ -327,7 +327,6 @@ bool matroska_segment_c::ParseSimpleTags( SimpleTag* pout_simple, KaxTagSimple *
     catch(...)
     {
         msg_Err( &sys.demuxer, "Error while reading Tag ");
-        delete ep;
         return false;
     }
 



More information about the vlc-commits mailing list