[Openvas-commits] r13714 - in trunk/openvas-libraries: . misc

[scm-commit at wald.intevation.org]

Author: jan
Date: 2012-07-10 22:45:40 +0200 (Tue, 10 Jul 2012)
New Revision: 13714

Modified:
   trunk/openvas-libraries/ChangeLog
   trunk/openvas-libraries/misc/store.c
Log:
* misc/store.c (store_plugin): Removed code that handles the
unused variable "path".



Modified: trunk/openvas-libraries/ChangeLog
===================================================================
--- trunk/openvas-libraries/ChangeLog	2012-07-10 17:23:12 UTC (rev 13713)
+++ trunk/openvas-libraries/ChangeLog	2012-07-10 20:45:40 UTC (rev 13714)
@@ -1,5 +1,10 @@
 2012-07-10  Jan-Oliver Wagner <jan-oliver.wagner at greenbone.net>
 
+	* misc/store.c (store_plugin): Removed code that handles the
+	unused variable "path".
+
+2012-07-10  Jan-Oliver Wagner <jan-oliver.wagner at greenbone.net>
+
 	* misc/plugutils.c (proto_post_wrapped): Resolved unnecessary
 	call of arg_get_value.
 

Modified: trunk/openvas-libraries/misc/store.c
===================================================================
--- trunk/openvas-libraries/misc/store.c	2012-07-10 17:23:12 UTC (rev 13713)
+++ trunk/openvas-libraries/misc/store.c	2012-07-10 20:45:40 UTC (rev 13714)
@@ -211,22 +211,11 @@
   gchar *dummy = g_build_filename (nvti_cache->cache_path, file, NULL);
   gchar *desc_file = g_strconcat (dummy, ".nvti", NULL);
   // assume there is a ".nvti" at the end in the cache path
-  gchar *path = g_strdup (file);
 
   g_free (dummy);
 
-  if (desc_file == NULL || path == NULL)
-    {
-      if (desc_file != NULL)
-        {
-          g_free (desc_file);
-        }
-      if (path != NULL)
-        {
-          g_free (path);
-        }
-      return;                   // g_build_filename failed
-    }
+  if (desc_file == NULL)
+    return;                   // g_build_filename failed
 
   nvti_t *n = arg_get_value (plugin, "NVTI");
   if (!n) n = nvti_new();
@@ -238,5 +227,4 @@
   arg_free_all (plugin);
 
   g_free (desc_file);
-  g_free (path);
 }