vlc | branch: master | Francois Cartegnie <[email protected]> | Thu May 16 
16:14:49 2019 +0200| [5d7bc4dbcb27cc74beb43c8eb0a2cf6369cdc4d7] | committer: 
Francois Cartegnie

demux: adaptive: fix potential use after free on oom

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

 modules/demux/adaptive/plumbing/FakeESOut.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/modules/demux/adaptive/plumbing/FakeESOut.cpp 
b/modules/demux/adaptive/plumbing/FakeESOut.cpp
index 4b2409cabf..42d46f3c34 100644
--- a/modules/demux/adaptive/plumbing/FakeESOut.cpp
+++ b/modules/demux/adaptive/plumbing/FakeESOut.cpp
@@ -137,8 +137,6 @@ FakeESOutID * FakeESOut::createNewID( const es_format_t 
*p_fmt )
         extrainfo->fillExtraFMTInfo( &fmtcopy );
 
     FakeESOutID *es_id = new (std::nothrow) FakeESOutID( this, &fmtcopy );
-    if(likely(es_id))
-        fakeesidlist.push_back( es_id );
 
     es_format_Clean( &fmtcopy );
 
@@ -324,6 +322,7 @@ es_out_id_t * FakeESOut::esOutAdd_Callback(es_out_t 
*fakees, const es_format_t *
         AbstractCommand *command = 
me->commandsqueue->factory()->createEsOutAddCommand( es_id );
         if( likely(command) )
         {
+            me->fakeesidlist.push_back(es_id);
             me->commandsqueue->Schedule( command );
             return reinterpret_cast<es_out_id_t *>(es_id);
         }

_______________________________________________
vlc-commits mailing list
[email protected]
https://mailman.videolan.org/listinfo/vlc-commits

Reply via email to