videostab/filter_videostab*.c: check for null

Fixes Coverity CID 709404:
Dereference before null check (REVERSE_INULL)
Dereferencing pointer "g". [show details]
 85        if ( !mlt_geometry_parse( g, vectors, length, -1, -1 ) )
...
Dereferencing "g" before a null check.
104        if ( g ) mlt_geometry_close( g );

diff --git a/src/modules/videostab/filter_videostab.c b/src/modules/videostab/filter_videostab.c
index c569801fb34260fc839c860f5789e78babec4612..995266614665530b8aded0a8f9a250fac20d983a 100644 (file)
--- a/src/modules/videostab/filter_videostab.c
+++ b/src/modules/videostab/filter_videostab.c
@@ -82,7 +82,7 @@ static void deserialize_vectors( videostab self, char *vectors, mlt_position len
        mlt_geometry g = mlt_geometry_init();
 
        // Parse the property as a geometry
-       if ( !mlt_geometry_parse( g, vectors, length, -1, -1 ) )
+       if ( g && !mlt_geometry_parse( g, vectors, length, -1, -1 ) )
        {
                struct mlt_geometry_item_s item;
                int i;

diff --git a/src/modules/videostab/filter_videostab2.c b/src/modules/videostab/filter_videostab2.c
index d6bd6855c71b190500577ebf0b5e911d7968eddf..ce76c7d864b84d5f51c46d8552b3dfefdb54de2c 100644 (file)
--- a/src/modules/videostab/filter_videostab2.c
+++ b/src/modules/videostab/filter_videostab2.c
@@ -82,7 +82,7 @@ Transform* deserialize_vectors( char *vectors, mlt_position length ,float scale_
        mlt_geometry g = mlt_geometry_init();
        Transform* tx=NULL;
        // Parse the property as a geometry
-       if ( !mlt_geometry_parse( g, vectors, length, -1, -1 ) )
+       if ( g && !mlt_geometry_parse( g, vectors, length, -1, -1 ) )
        {
                struct mlt_geometry_item_s item;
                int i;