[PATCH] Fix 53360 - String leak in DCOPClient
ravi at ee.eng.ohio-state.edu
Sun Mar 9 00:19:03 GMT 2003
-----BEGIN PGP SIGNED MESSAGE-----
> Are you working with HEAD?
> The patch is against CVS HEAD, but it seems unnecessary given coolo's
> commit on 11 Feb which is almost the same as yours.
My mistake. I had fixed this a while ago, and when I updated CVS, I missed
that this file was not updated. Sorry.
About the patch of Luis: I would still check that the local variables are not
NULL before freeing them (paranoia, I presume :-) That said, I like your
patch better. In case we stick with Coolo's patch, I would like to make sure
that we initialize the variables to 0L in the constructor (paranoia again).
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)
-----END PGP SIGNATURE-----
More information about the kde-core-devel