Re: [PATCH] Attachments in mailto: URLs

[Peter Bloomfield <peterbloomfield bellsouth net>]

On 11/01/2006 08:33:30 AM Wed, Blackwell, Steve @ GSI wrote:
> Seems to me that a memory leak has been introduced here. If
> add_attachment()
> is called, then g_free(val) is not but "val = decode_and_strdup(ptr,'&',
> &ptr);" is called every time.
>
> Steve

Hi Steve:

That's because add_attachment() steals the string.  The version in cvs  
moves the test to sendmsg_window_set_field(), so key is always freed in  
sendmsg_window_process_url(), but a new copy is passed to add_attachment()  
in sendmsg_window_set_field().

Peter