mysql_selector.c: free(NULL) is OK.
authorAndre Noll <maan@systemlinux.org>
Mon, 29 Jan 2007 22:47:37 +0000 (23:47 +0100)
committerAndre Noll <maan@systemlinux.org>
Mon, 29 Jan 2007 22:47:37 +0000 (23:47 +0100)
No need to check for NULL. Moreover, stream, meta, atts and dir are
set in any case so kill the NULL initialization.

mysql_selector.c

index e2812fd..91972e8 100644 (file)
@@ -766,7 +766,7 @@ out:
  */
 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;
 
@@ -792,14 +792,10 @@ write:
                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;