Re: [Openvas-devel] [Openvas-commits] r6081 - in trunk/openvas-manager: . src

[Matthew Mundell]

> Log:
> Fixed up some free()s

> --- trunk/openvas-manager/src/omp.c   2009-12-07 14:20:51 UTC (rev 6080)
> +++ trunk/openvas-manager/src/omp.c   2009-12-07 15:17:12 UTC (rev 6081)
> @@ -5414,7 +5414,7 @@
>                if (hosts == NULL)
>                  {
>                    request_delete_task (&current_client_task);
> -                  g_free (description);
> +                  free (description);
>                    free (tsk_uuid);
>                    SEND_TO_CLIENT_OR_FAIL
>                     (XML_ERROR_SYNTAX
> @@ -5424,7 +5424,7 @@
>                    set_client_state (CLIENT_AUTHENTIC);
>                    break;
>                  }
> -              g_free (description);
> +              free (description);
>
>                target_name = g_strdup_printf ("Imported target for task %s",
>                                               tsk_uuid);
> @@ -5947,7 +5947,7 @@
>                      g_free (response);
>                      g_free (name);
>                      g_free (description64);
> -                    g_free (tsk_uuid);
> +                    free (tsk_uuid);

Thanks Tim, these three are actually allocated by GLib.

The Manager mixes C and GLib allocations extensively, mostly because I've
been reluctant to put GLib in any interface that could become external.  I
think the mixing is safe for now.  I'd like to clean through it all at some
point.
--
Intevation GmbH, Neuer Graben 17, 49074 Osnabrück | AG Osnabrück, HR B 18998
Geschäftsführer: Frank Koormann, Bernhard Reiter, Dr. Jan-Oliver Wagner
_______________________________________________
Openvas-devel mailing list
Openvas-devel@wald.intevation.org
http://lists.wald.intevation.org/mailman/listinfo/openvas-devel