[vlc-devel] commit: video_filter_rss: use the return value and fix mutex leak ( the mutex was never ( Rémi Duraffort )

[git version control]

vlc | branch: master | Rémi Duraffort <ivoire at videolan.org> | Tue Sep  1 10:41:06 2009 +0200| [352188c2d9345da14ad4708a3c3c141ca21520d9] | committer: Rémi Duraffort 

video_filter_rss: use the return value and fix mutex leak (the mutex was never
used but it will be used soon).

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

 modules/video_filter/rss.c |    7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/modules/video_filter/rss.c b/modules/video_filter/rss.c
index 5fb377e..e13137a 100644
--- a/modules/video_filter/rss.c
+++ b/modules/video_filter/rss.c
@@ -281,7 +281,11 @@ static int CreateFilter( vlc_object_t *p_this )
     }
 
     /* Parse the urls */
-    ParseUrls( p_filter, psz_urls );
+    if( ParseUrls( p_filter, psz_urls ) )
+    {
+        free( psz_urls );
+        goto error;
+    }
     free( psz_urls );
 
     if( FetchRSS( p_filter ) )
@@ -328,6 +332,7 @@ static void DestroyFilter( vlc_object_t *p_this )
     filter_t *p_filter = (filter_t *)p_this;
     filter_sys_t *p_sys = p_filter->p_sys;
 
+    vlc_mutex_destroy( &p_sys->lock );
     text_style_Delete( p_sys->p_style );
     free( p_sys->psz_marquee );
     FreeRSS( p_filter );