KDE/kdelibs/kio/kio

[David Faure]

SVN commit 550847 by dfaure:

QGlobalStatic destruction happens -before- KStaticDeleter :(
==12871== Invalid read of size 8
==12871==    at 0x4C1F9A9: QTextCodec::fromUnicode(QString const&) const (qtextcodec.cpp:978)
==12871==    by 0x4B9B301: QString::toLocal8Bit() const (qstring.cpp:2956)
==12871==    by 0x4FD1E2E: kdbgstream::flush() (kdebug.cpp:406)
==12871==    by 0x4FCEB4D: kdbgstream::operator<<(QString const&) (kdebug.cpp:459)
==12871==    by 0x40ADE7: kdbgstream::operator<<(char const*) (kdebug.h:224)
==12871==    by 0x5772E97: endl(kdbgstream&) (kdebug.h:401)
==12871==    by 0x4FCF95B: kdbgstream::operator<<(kdbgstream& (*)(kdbgstream&)) (kdebug.cpp:512)
==12871==    by 0x5770E78: KDirWatchPrivate::~KDirWatchPrivate() (kdirwatch.cpp:194)
==12871==    by 0x5770CA6: KDirWatch::~KDirWatch() (kdirwatch.cpp:1222)
==12871==    by 0x5774E74: KStaticDeleter<KDirWatch>::destructObject() (kstaticdeleter.h:143)
==12871==    by 0x5774F01: KStaticDeleter<KDirWatch>::~KStaticDeleter() (kstaticdeleter.h:153)
==12871==    by 0x577266F: __tcf_0 (kdirwatch.cpp:1183)
==12871==    by 0x85F6D5A: __cxa_finalize (in /lib/libc-2.3.6.so)
==12871==    by 0x576D502: (within /d/kde/build/4/kdelibs/lib/libkio.so.5.0.0)
==12871==    by 0x590B9D0: (within /d/kde/build/4/kdelibs/lib/libkio.so.5.0.0)
==12871==    by 0x85F6A74: exit (in /lib/libc-2.3.6.so)
==12871==    by 0x85E14A1: __libc_start_main (in /lib/libc-2.3.6.so)
==12871==  Address 0x980D2A0 is 0 bytes inside a block of size 8 free'd
==12871==    at 0x4A1A84C: operator delete(void*) (vg_replace_malloc.c:246)
==12871==    by 0x4C22A44: QUtf8Codec::~QUtf8Codec() (qutfcodec.cpp:31)
==12871==    by 0x4C2041F: QTextCodecCleanup::~QTextCodecCleanup() (qtextcodec.cpp:163)
==12871==    by 0x4C22250: QGlobalStatic<QTextCodecCleanup>::~QGlobalStatic() (qglobal.h:1236)
==12871==    by 0x4C2173F: __tcf_1 (qtextcodec.cpp:170)
==12871==    by 0x85F6A74: exit (in /lib/libc-2.3.6.so)
==12871==    by 0x85E14A1: __libc_start_main (in /lib/libc-2.3.6.so)
CCMAIL: kde-core-devel at kde.org


 M  +1 -2      kdirwatch.cpp  


--- trunk/KDE/kdelibs/kio/kio/kdirwatch.cpp #550846:550847
@@ -180,7 +180,7 @@
   kDebug(7001) << "Available methods: " << available << endl;
 }
 
-/* This should never be called, but doesn't harm */
+/* This is called on app exit (KStaticDeleter) */
 KDirWatchPrivate::~KDirWatchPrivate()
 {
   timer.stop();
@@ -191,7 +191,6 @@
 #ifdef HAVE_FAM
   if (use_fam) {
     FAMClose(&fc);
-    kDebug(7001) << "KDirWatch deleted (FAM closed)" << endl;
   }
 #endif
 #ifdef HAVE_SYS_INOTIFY_H