[vlc-devel] commit: Simplify: false positive CID 139 ( Rafaël Carré )

git version control git at videolan.org
Fri May 30 14:18:14 CEST 2008


vlc | branch: master | Rafaël Carré <funman at videolan.org> | Fri May 30 14:08:19 2008 +0200| [60d602251a048ac8160867a00a8f56165c5c34b2]

Simplify: false positive CID 139

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

 modules/codec/dvbsub.c |    9 ++-------
 1 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/modules/codec/dvbsub.c b/modules/codec/dvbsub.c
index 6424859..ddb011e 100644
--- a/modules/codec/dvbsub.c
+++ b/modules/codec/dvbsub.c
@@ -870,14 +870,9 @@ static void decode_region_composition( decoder_t *p_dec, bs_t *s )
 
     /* Free old object defs */
     while( p_region->i_object_defs )
-    {
-        int i = p_region->i_object_defs - 1;
-        free( p_region->p_object_defs[i].psz_text );
-        if( !i )
-            free( p_region->p_object_defs );
+        free( p_region->p_object_defs[--p_region->i_object_defs].psz_text );
 
-        p_region->i_object_defs--;
-    }
+    free( p_region->p_object_defs );
     p_region->p_object_defs = NULL;
 
     /* Extra sanity checks */




More information about the vlc-devel mailing list