[PATCH] Fix 53360 - String leak in DCOPClient
Ravikiran Rajagopal
ravi at ee.eng.ohio-state.edu
Sun Mar 9 00:19:03 GMT 2003
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
> 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).
Any thoughts?
Ravi
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)
iD8DBQE+aoh3bI8Y8y0oVXcRAte9AJwK3fvUfBfwSVbY5T8q8y5zeGGvGgCeMfXO
V1GvGCagFZgXi4OcS2HgOhs=
=qUcm
-----END PGP SIGNATURE-----
More information about the kde-core-devel
mailing list