Ticket #3961 (new defect)

Opened 5 years ago

Last modified 5 years ago

Move over ssh shell will destory files when target FS full

Reported by: BruXy Owned by:
Priority: critical Milestone: Future Releases
Component: mc-vfs Version: 4.8.15
Keywords: shell, ssh, mv Cc: howaboutsynergy@…, info@…
Blocked By: #77 Blocking:
Branch state: no branch Votes for changeset:

Description

Happened in mc 4.8.15-2 in Ubuntu 16.04.5 LTS.

  1. I was moving huge amount of files over ssh shell (Shell link).
  2. The target file system became full.
  3. When 2. happened files were still copied but appeared on target with zero sizes.
  4. Files were deleted on the source.

I would expect an error message and not deleting source files.

Change History

comment:1 Changed 5 years ago by BruXy

  • Version changed from master to 4.8.15
mc --version
GNU Midnight Commander 4.8.15
Built with GLib 2.47.3
Using the S-Lang library with terminfo database
With builtin Editor
With subshell support as default
With support for background operations
With mouse support on xterm and Linux console
With support for X11 events
With internationalization support
With multiple codepages support
Virtual File Systems: cpiofs, tarfs, sfs, extfs, ext2undelfs, ftpfs, sftpfs, fish
Data types: char: 8; int: 32; long: 64; void *: 64; size_t: 64; off_t: 64;

comment:2 Changed 5 years ago by andvaranaut

Hi, please check #3963, which seems to be another facet of the same bug, and which, as I have confirmed, is still present in the current stable version (4.8.22).

comment:3 Changed 5 years ago by BruXy

Yes, it may have the same root cause as #3963 and this ticket is duplicating it.

comment:4 Changed 5 years ago by howaboutsynergy

  • Cc howaboutsynergy@… added

comment:5 Changed 5 years ago by metux

  • Cc info@… added

comment:6 Changed 5 years ago by andrew_b

  • Blocked By 77 added

comment:7 Changed 5 years ago by metux

Sure that #77 is really necessary for that ?

I believe it should just check whether writing to remote side finished properly.
(maybe do a stat afterwards), before unlink'ing the original.

--mtx

Note: See TracTickets for help on using tickets.