Summary: | Crash using back button after opening a file in find result listing | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Andrea Aime <aaime> |
Component: | general | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED DUPLICATE | ||
Severity: | crash | CC: | kollix |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Other | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Andrea Aime
2002-04-25 09:16:55 UTC
--=.iBfA44cbTpKn5A Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Package: konqueror kfind Version: KDE_3_0_2_RELEASE OS: Linux 2.4.18 Hi After a sleepless night I have patch for this bug :) It seems that "emit finished()" causes the crashes. What connects to this signal? I couldn't find anything significant. Backtrace shows that it causes openUrl to be called with some strange parameters. As far as I know commenting it out causes no side effects. Change in libkonq also fixes strange behavior when find tool gets closed if back button is pushed. Best regards Tomaz Solc Index: kfind/kfindpart.cpp =================================================================== RCS file: /home/kde/kdebase/kfind/kfindpart.cppv retrieving revision 1.4 diff -u -3 -p -r1.4 kfindpart.cpp --- kfind/kfindpart.cpp25 Feb 2002 12:16:32 -00001.4 +++ kfind/kfindpart.cpp16 Aug 2002 10:18:31 -0000 @@ -1677 +1679 @@ void KFindPart::restoreKFindState( QData *stream >> itemUrl; addFile(new KFileItem(itemUrl""0)""); } - emit finished(); + // why does this make konqueror crash? + //emit finished(); + kdDebug(1202) << "KFindPart::restoreKFindState : would emit finished\n"; } KFindPartBrowserExtension::KFindPartBrowserExtension( KFindPart *findPart ) Index: libkonq/konq_dirpart.cc =================================================================== RCS file: /home/kde/kdebase/libkonq/konq_dirpart.ccv retrieving revision 1.54 diff -u -3 -p -r1.54 konq_dirpart.cc --- libkonq/konq_dirpart.cc25 Feb 2002 06:38:55 -00001.54 +++ libkonq/konq_dirpart.cc16 Aug 2002 10:18:33 -0000 @@ -2506 +25013 @@ void KonqDirPart::restoreNameFilter( QDa void KonqDirPart::restoreState( QDataStream &stream ) { + if ( m_findPart ) + { + delete m_findPart; + m_findPart = 0L; + emit findClosed( this ); + } + // Warning: see comment in IconViewBrowserExtension::restoreState about order //kdDebug(1203) << " -- void KonqDirPart::restoreState( QDataStream &stream )" << endl; bool bFindPart; @@ -46014 +46716 @@ bool KonqDirPart::closeURL() bool KonqDirPart::openURL(const KURL& url) { - if ( m_findPart ) - { - kdDebug(1203) << "KonqDirPart::openURL -> emit findClosed " << this << endl; - delete m_findPart; - m_findPart = 0L; - emit findClosed( this ); - } - +// if ( m_findPart ) +// { +// kdDebug(1203) << "KonqDirPart::openURL -> emit findClosed " << this << endl; +//kdDebug(1203) << "KonqDirPart::openURL url: " << url.path() << endl; +// delete m_findPart; +// m_findPart = 0L; +// emit findClosed( this ); +// } +// moved to restore state + m_url = url; emit aboutToOpenURL (); --=.iBfA44cbTpKn5A Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.0.6 (GNU/Linux) iD8DBQE9XNucsAlAlRhL9q8RAowNAKDL/tDgd973O4yMzHBjapoCAgepUwCgrEiQ SXcwqeD1EkYWsdCOVtqQ5xU= =nADD -----END PGP SIGNATURE----- --=.iBfA44cbTpKn5A-- *** Bug 42337 has been marked as a duplicate of this bug. *** *** This bug has been marked as a duplicate of 47503 *** this bug is still not fixed, i downloaded the latest version of konqueror just now, and it still does this. Jeff |