tis 2002-08-27 klockan 19.09 skrev Michael Cardenas:
> Here's a cleaner version of this patch. 

Dang. I should read all my mail before starting to reply to some of it
:-)

> +char *partkit_get_devices(PedDevice *dev)
[snip]

This fixes some of my comments in the other mail.

> +      ptr = partkit_get_devices(dev);
>  
>        client->command (client, "subst", "partkit/select_device", "choices",
>                      ptr, NULL);
> +      free(ptr);
> +      
>        /*FIXME: how to get a default device? */
>        client->command (client, "subst", "partkit/select_device", "default",
>                      ptr, NULL);

But you're still using ptr after it's been freed :-)

Attachment: signature.asc
Description: Detta =?ISO-8859-1?Q?=E4r?= en digitalt signeradmeddelandedel

Reply via email to