mysql_selector.c: free(NULL) is OK.

No need to check for NULL. Moreover, stream, meta, atts and dir are
set in any case so kill the NULL initialization.

diff --git a/mysql_selector.c b/mysql_selector.c
index e2812fd7904df3d155d422569b74c86683bc4669..91972e84ab0c179fba33129a811d171bd1c00021 100644 (file)
--- a/mysql_selector.c
+++ b/mysql_selector.c
@@ -766,7 +766,7 @@ out:
  */
 static char *get_selector_info(char *name)
 {
  */
 static char *get_selector_info(char *name)
 {

-       char *meta = NULL, *atts = NULL, *info, *dir = NULL, *query, *stream = NULL;
+       char *meta, *atts, *info, *dir, *query, *stream;

        void *result = NULL;
        MYSQL_ROW row = NULL;
 
        void *result = NULL;
        MYSQL_ROW row = NULL;
 


@@ -792,14 +792,10 @@ write:
                stream, meta, 
                (result && row && row[0])? row[0] : "(no score)",
                atts);
                stream, meta, 
                (result && row && row[0])? row[0] : "(no score)",
                atts);

-       if (dir)
-               free(dir);
-       if (meta)
-               free(meta);
-       if (atts)
-               free(atts);
-       if (stream)
-               free(stream);
+       free(dir);
+       free(meta);
+       free(atts);
+       free(stream);

        if (result)
                mysql_free_result(result);
        return info;
        if (result)
                mysql_free_result(result);
        return info;