Bug 56840 - Konqueror tortures X11 on Out Of Disk space
Summary: Konqueror tortures X11 on Out Of Disk space
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: bookmarks (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR crash
Target Milestone: ---
Assignee: Konqueror Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-04-04 18:28 UTC by Charles Samuels
Modified: 2003-07-06 16:00 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Charles Samuels 2003-04-04 18:28:15 UTC
Version:            (using KDE Devel)
Installed from:    Compiled sources
OS:          Linux

If you run out of disk space, and visit a web site, it'll say "can't write bookmarks.xml - out of disk space".  Problem is that it shows this in a dialog box, and it will show an endless supply of these dialog boxes, thereby DoSing your X server, and pissing off kwin & kicker's taskbar in the process.  I think you should just ignore this problem silently.

Why does it write to the bookmarks file when you're just browsing, anyway?
Comment 1 lypanov 2003-07-06 16:00:41 UTC
Subject: kdelibs/kio/bookmarks

CVS commit by lypanov: 

Only show "can't write bookmarks.xml - out of disk space" 
once to prevent DoS'ing of the users screen. Also improve
the error message slightly, though i guess it could be
possible duplication of data: "out of disk space"
CCMAIL: 56840-done@bugs.kde.org


  M +11 -1     kbookmarkmanager.cc   1.65


--- kdelibs/kio/bookmarks/kbookmarkmanager.cc  #1.64:1.65
@@ -265,5 +265,15 @@ bool KBookmarkManager::saveAs( const QSt
 
 failure:
-    KMessageBox::error( 0L, i18n("Couldn't save bookmarks in %1. %2").arg(filename).arg(strerror(file.status())) );
+    static int hadSaveError = false;
+    if ( !hadSaveError )
+        KMessageBox::error( 
+            0L, i18n("Couldn't save bookmarks in %1. Reported error was: %2."
+                     "This error message will only be shown once, please fix the "
+                     "possible cause of the error as quickly as possible, "
+                     "the most likely cause is that of a full harddrive")
+                .arg(filename)
+                .arg(strerror(file.status())) 
+        );
+    hadSaveError = true;
     return false;
 }