Deleting files from Incomplete DL folder corrups DL's

Bug #225592 reported by Sparky55
2
Affects Status Importance Assigned to Milestone
DC++
Fix Released
Medium
Unassigned

Bug Description

If incomplete files are deleted from the Incomplete DL folder, when DC++ 0.705 resumes downloading of that file, a file gets generated that seems to be the size of the deleted file. I am having a large number of larger .AVI files failing to work. This Bug would enplane this problem.

I noticed this when I cleared out the Incomplete DL folder, and started up DC++ V0.705, and a few downloads started from a resumed state, and within a few seconds a 22 MB file appeared. Another resumed a minute later and a 220 MB file appeared! Checking back a short while later another 390 MB file was there. Within 10 minutes, 1.33 GB was in the download folder, after the folder was empty, which is impossible for my cable connection to be able to download.

Previous versions of DC++ (0.6xx and earlier) would just start the file over from the beginning, which is expected. The incomplete folder needs the files cleared out sometimes, since incomplete downloads will eventually consume my entire hard drive partition!

OS: Win 2KPRo running DC++ 0.705

Revision history for this message
Sparky55 (levy-jeff) wrote :

Also note that I an NOT using the antifrag option, although the files have .antifrag at the end.

Revision history for this message
Sparky55 (levy-jeff) wrote :

Opening the 22 MB file, which is now a 28 MB file, up in a Hex editor, the first 22 MB of the file contains just 0x00!!! The remainder of the file contains actual data (~6 MB).

Revision history for this message
Sparky55 (levy-jeff) wrote :

Attached is a corrupted file

Revision history for this message
eMTee (realprogger) wrote :

Try it with 0.706 as probably it is fixed in the latest release (antifragmentation method is mandatory in 0.706 and cannot be disabled from now).

Revision history for this message
Jacek Sieka (arnetheduck) wrote :

actually, no, there's no check done if the temp file exists when resuming, so it's plausible that it's filled with 0's...the difference will be that in 0.706 the whole file will be recreated...

Changed in dcplusplus:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Jacek Sieka (arnetheduck) wrote :

committed fix needs testing...

Changed in dcplusplus:
status: Confirmed → Fix Committed
Revision history for this message
eMTee (realprogger) wrote :

tested, bzr1276 fixed this...

MikeJJ (mrmikejj)
Changed in dcplusplus:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.