Closed
Bug 48620
Opened 24 years ago
Closed 24 years ago
Out of diskspace condition not handled correctly in FTP downloads
Categories
(SeaMonkey :: UI Design, defect, P2)
Tracking
(Not tracked)
People
(Reporter: admin, Assigned: law)
References
()
Details
(Keywords: crash, Whiteboard: [nsbeta3-][PDTP2][rtm need info]Fix in hand)
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; m17) Gecko/20000807
BuildID: 2000080712
I wanted to save a file that was too big to save on a floppy (the URL above is
~3.0 MB). Instead of warning me that the file wouldn't fit on the disk, it just
downloaded as normal and attempted to save anyway
Reproducible: Didn't try
Comment 1•24 years ago
|
||
-> Networking? How do we handle out-of-disk-space or low-disk-space conditions?
Gerv
Component: Event Handling → Networking
Comment 4•24 years ago
|
||
Checked the winNT build 2000082208. If the user proceeds to download the file
and he doesn't have enough disk space, he will see an 'unknown error 2 80004005'
dialog and then crash. Marking nsBeta3 for review.
Should this be a warning or should we prevent the save from happening?
Keywords: nsbeta3
OS: Windows 98 → All
Comment 6•24 years ago
|
||
Nav triage team: [nsbeta3+], adding crash keyword
Keywords: crash
Whiteboard: [nsbeta3+]
see probable dupe of this: bug 50697 "Solaris/Linux crash when saving file if
over disk quota/out of disk space"
Assignee | ||
Comment 10•24 years ago
|
||
These patches will avoid a crash. There's still work to do if we actually want
to warn and handle the problem gracefully.
Index: nsStreamXferOp.cpp
===================================================================
RCS file: /cvsroot/mozilla/xpfe/components/xfer/src/nsStreamXferOp.cpp,v
retrieving revision 1.32
diff -w -r1.32 nsStreamXferOp.cpp
330a331
> break;
333a335
> break;
Index: base/src/nsBufferedStreams.cpp
> ===================================================================
> RCS file: /cvsroot/mozilla/netwerk/base/src/nsBufferedStreams.cpp,v
> retrieving revision 1.8
> diff -r1.8 nsBufferedStreams.cpp
> 335a336,339
> > if (!mStream) {
> > // Stream already cancelled/flushed; probably because of error.
> > return NS_OK;
> > }
Comment 11•24 years ago
|
||
Let's get both problems fixed in RTM, if poosible.
Comment 12•24 years ago
|
||
Changing the summary to make it more descriptive, and adding relnote3 keyword.
Keywords: relnote3
Summary: Didn't warn about file size → Out of diskspace condition not handled correctly in FTP downloads
Comment 13•24 years ago
|
||
PDT marking [rtm need info] since no code reviews are listed yet.
Whiteboard: [nsbeta3-][PDTP2][rtm+] → [nsbeta3-][PDTP2][rtm need info]
Assignee | ||
Comment 14•24 years ago
|
||
I've posted this attachment to bug 35161 to fix that bug and this one (and also
50697):
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=16135
Comment 15•24 years ago
|
||
Fix in hand.
Whiteboard: [nsbeta3-][PDTP2][rtm need info] → [nsbeta3-][PDTP2][rtm need info]Fix in hand
Assignee | ||
Comment 16•24 years ago
|
||
Closing as dup of 35161. I've fixed the crash and improved the message.
Still no advance warning if you're going to run out of space. That will have to
wait for the next train.
This applies only in the "Save link as..." scenario, BTW.
*** This bug has been marked as a duplicate of 35161 ***
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → DUPLICATE
Comment 18•23 years ago
|
||
-> xpapps.
RELNOTE patrol...
Worth noting that this is duped to a bug that doesn't entirely sound the same,
so does this need to be set to relnote and documented?
Component: Networking → XP Apps
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•