[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    Bug#17414: KDE bug report logs - #17414 dcop crash dcopclient + fix
From:       Simon Edwards <simon () simonzone ! com>
Date:       2001-09-27 18:51:07
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


Hi,

> Would instead of
>         IceCloseConnection(d->iceConn);
>         d->iceConn = 0L;    // <-------- Fixed.
>
> calling detach() itself not be more correct?

I don't know. You're stretching my memory, but I think somewhere else code 
tries to clean up d->iceConn, sees the non-null pointer, tries to free and 
ends up segfaulting due to double free()'ing memory somewhere.

Anyway, it's usually a good idea to null pointers when they are no longer 
valid.

cheers,


- -- 
Simon Edwards
simon@simonzone.com
http://www.simonzone.com/
Nijmegen, The Netherlands       "ZooTV? You made the right choice."
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.6 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iEYEARECAAYFAjuzdRsACgkQuIuDmTrvhSbutwCZAcaMWlBdL8YvvOtYgj7yw/pR
IVIAnRHlabA2+a0ax26UdpaeFkxOCYXN
=EprM
-----END PGP SIGNATURE-----

[prev in list] [next in list] [prev in thread] [next in thread]