Fixed order between xmlFreeTextReader and xmlCleanupParser.

 I am not sure it had consequences, but it seems more logical and
it follows the libxml2 code example I found (reader1.c).

diff --git a/modules/misc/xml/libxml.c b/modules/misc/xml/libxml.c
index d448cfcf487bbbf4b337117a3715026310d35db0..47b1f782c823f890cbacbc9dbf90cb74b1d08f4d 100644 (file)
--- a/modules/misc/xml/libxml.c
+++ b/modules/misc/xml/libxml.c
@@ -179,12 +179,12 @@ static void ReaderClose( vlc_object_t *p_this )
 {
     xml_reader_t *p_reader = (xml_reader_t *)p_this;
 
+    xmlFreeTextReader( (void *)p_reader->p_sys );
 #ifdef LIBXML_GETS_A_CLUE_ABOUT_REENTRANCY_AND_MEMORY_LEAKS
     vlc_mutex_lock( &lock );
     xmlCleanupParser();
     vlc_mutex_unlock( &lock );
 #endif
-    xmlFreeTextReader( (void *)p_reader->p_sys );
 }
 
 static int ReaderUseDTD ( xml_reader_t *p_reader )