[PATCH] Fix 53360 - String leak in DCOPClient

Waldo Bastian bastian at kde.org
Sun Mar 9 00:14:12 GMT 2003


On Sunday 09 March 2003 00:55, Luis Pedro Coelho wrote:
> Why do we even have DCOPClientPrivate::vendor and
> DCOPClientPrivate::release anyway? I propose the following patch:

Looks good, except that

> +    free(vendor);
> +    free(release);

should be if (vendor) free(vendor); because free doesn't like to free 0 
pointers, unlike delete.

Cheers,
Waldo
-- 
bastian at kde.org -=|[ SuSE, The Linux Desktop Experts ]|=- bastian at suse.com





More information about the kde-core-devel mailing list