Clean up possibly-null pointer dereference (#4812).

git-svn-id: svn://localhost/ardour2/branches/3.0@11813 d708f5d6-7413-0410-9779-e7cbd77b26cf

diff --git a/gtk2_ardour/sfdb_freesound_mootcher.cc b/gtk2_ardour/sfdb_freesound_mootcher.cc
index a1bffecfebfbe07f3cda22e70ec47c29fee66196..7e13c4f68b285818ec90b23ae13e95898d1654d6 100644 (file)
--- a/gtk2_ardour/sfdb_freesound_mootcher.cc
+++ b/gtk2_ardour/sfdb_freesound_mootcher.cc
@@ -187,17 +187,16 @@ std::string Mootcher::doRequest(std::string uri, std::string params)
                return "";\r
        }\r
 \r
-       result = xml_page.memory;\r
        // free the memory\r
-       if(xml_page.memory){\r
-               free( xml_page.memory );\r
-               xml_page.memory = NULL;\r
-               xml_page.size = 0;\r
+       if (xml_page.memory) {\r
+               result = xml_page.memory;\r
        }\r
+       \r
+       free (xml_page.memory);\r
+       xml_page.memory = NULL;\r
+       xml_page.size = 0;\r
 \r
-//printf("freesound result: %s\n", result.c_str());\r
        return result;\r
-\r
 }\r
 \r
 \r