Bug 54613

Summary:	Expanding a sidebar URL causes a crash if the URL doesn't end in a slash
Product:	[Applications] konqueror	Reporter:	Lenon Kitchens <lenon>
Component:	general	Assignee:	Konqueror Bugs <konqueror-bugs-null>
Status:	RESOLVED FIXED
Severity:	crash	CC:	2rparlett, fred, lehman, r.amore, thubi
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Lenon Kitchens 2003-02-13 23:49:09 UTC

Version:            (using KDE KDE 3.1)
Installed from:    Compiled From Sources
Compiler:          gcc version 3.2 (Mandrake Linux 9.0 3.2-1mdk) 
OS:          Linux

On the "Network" tab of the sidebar I created a new item under ftp archives.  I did this by copying the "KDE Official FTP" item, pasting and modifying it, and then copying the new version back to the sidebar under FTP Archives.  The only 2 things I modified were Name and URL.  The "Name" I set to NMS Server (Production).  The URL I set to "ftp://wmerpt01.arch.com".  At this point I could click on the item and get a login prompt and then konqueror would list the files correctly.  However, if I clicked on the plus beside the item konqueror would crash.  I finally got it to quit crashing by setting the URL to "ftp://wmerpt01.arch.com/export/home/user".

Comment 1 John Firebaugh 2003-04-07 05:50:43 UTC

*** Bug 56880 has been marked as a duplicate of this bug. ***

Comment 2 John Firebaugh 2003-04-07 05:52:28 UTC

*** Bug 53450 has been marked as a duplicate of this bug. ***

Comment 3 John Firebaugh 2003-04-07 05:52:32 UTC

*** Bug 53548 has been marked as a duplicate of this bug. ***

Comment 4 Thiago Macieira 2003-05-04 00:43:00 UTC

*** Bug 58086 has been marked as a duplicate of this bug. ***

Comment 5 Stephan Kulow 2003-05-05 15:27:27 UTC

Subject: kdebase/konqueror/sidebar/trees/dirtree_module

CVS commit by coolo: 

do not up cast to invalid classes (I bet this is also the cause of many lan:/ crashes
we've seen)
CCMAIL: 54613-done@bugs.kde.org


  M +9 -5      dirtree_module.cpp   1.25


--- kdebase/konqueror/sidebar/trees/dirtree_module/dirtree_module.cpp  #1.24:1.25
@@ -135,9 +135,10 @@ void KonqSidebarDirTreeModule::removeSub
     {
         bool b = m_dictSubDirs.remove( item->externalURL().url(-1) );
-        while (!(((KonqSidebarDirTreeItem*) item)->alias.isEmpty()))
+        KonqSidebarDirTreeItem *ditem = dynamic_cast<KonqSidebarDirTreeItem*>(item);
+        while (ditem && !(ditem->alias.isEmpty()))
         {
-                b = b|m_dictSubDirs.remove( ((KonqSidebarDirTreeItem*) item)->alias.front() );
+                b = b|m_dictSubDirs.remove( ditem->alias.front() );
 
-                ((KonqSidebarDirTreeItem*) item)->alias.pop_front();
+                ditem->alias.pop_front();
         }
         if (!b)
@@ -315,6 +316,9 @@ void KonqSidebarDirTreeModule::slotRedir
         if (item->parent())
         {
-            ((KonqSidebarDirTreeItem*) item)->alias<<oldUrl.url(-1);
-            ((KonqSidebarDirTreeItem*) item)->alias<<newUrl.url(-1);
+            KonqSidebarDirTreeItem* ditem = dynamic_cast<KonqSidebarDirTreeItem*>(item);
+            if (ditem) {
+                ditem->alias<<oldUrl.url(-1);
+                ditem->alias<<newUrl.url(-1);
+            }
         }
         kdDebug(1201) << "Updating url to " << newUrl.prettyURL() << endl;

Comment 6 Stephan Kulow 2003-05-06 09:39:41 UTC

*** Bug 44939 has been marked as a duplicate of this bug. ***