[vlc-devel] commit: Fix potential dereference of already free()d pointer (David Flynn )

git version control git at videolan.org
Fri Jul 11 20:14:05 CEST 2008


vlc | branch: master | David Flynn <davidf at rd.bbc.co.uk> | Fri Jul 11 18:04:26 2008 +0000| [fe9b4d25e1359047110fd913d93707e7d2c142b2]

Fix potential dereference of already free()d pointer

In some error cases p_block will have been freed before p_block is
dereferenced.

Signed-off-by: Rémi Denis-Courmont <rdenis at simphalempin.com>

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

 modules/codec/schroedinger.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/modules/codec/schroedinger.c b/modules/codec/schroedinger.c
index adf8017..de41b73 100644
--- a/modules/codec/schroedinger.c
+++ b/modules/codec/schroedinger.c
@@ -469,11 +469,6 @@ static picture_t *DecodeBlock( decoder_t *p_dec, block_t **pp_block )
             if( b_bail )
                 break;
         }
-
-        if( (i_bufused > 0) && (i_bufused < p_block->i_buffer) )
-            msg_Err( p_dec, "not submitted everything in p_buffer");
-
-
     } while( 0 );
 
     while( 1 )




More information about the vlc-devel mailing list