bleachbit / bleachbit / 22011811711

71%

push

github

Fix cluster overrun in wipe_extent_by_defrag causing move errors

The temporary zero-fill file could be allocated more clusters than the
target extent required (e.g., due to NTFS allocation granularity or
fragmentation). The code then attempted to move all of those clusters
into the target range, overshooting the end of the extent and causing
FSCTL_MOVE_FILE to fail.

Cap the number of clusters moved to the remaining target extent size.
Also log the exception details on move failure.

Takeaway: Wiping files (with the overwriting option enabled) with
administrator privileges on Microsoft Windows will be faster now than
it was in version 5.0.2 (stable).

Closes https://github.com/bleachbit/bleachbit/issues/1938

7 of 10 new or added lines in 1 file covered. (70.0%)

6320 of 8857 relevant lines covered (71.36%)

0.71 hits per line