[Bug 856501] New: mc dataloss: moving a file and skipping upon problem deletes original file
https://bugzilla.novell.com/show_bug.cgi?id=856501 https://bugzilla.novell.com/show_bug.cgi?id=856501#c0 Summary: mc dataloss: moving a file and skipping upon problem deletes original file Classification: openSUSE Product: openSUSE 13.1 Version: Final Platform: Other OS/Version: Other Status: NEW Severity: Critical Priority: P5 - None Component: Other AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: richlv@nakts.net QAContact: qa-bugs@suse.de Found By: --- Blocker: --- User-Agent: Opera/9.80 (X11; Linux x86_64) Presto/2.12.388 Version/12.15 in midnight commander, move a file to some location that causes a problem (for example, unable to chown it on the other side over ssh/shell link). when mc complains, choose 'skip'. target file is not closed, but original file is still deleted. Reproducible: Always -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=856501
https://bugzilla.novell.com/show_bug.cgi?id=856501#c1
--- Comment #1 from Rihards Olups
https://bugzilla.novell.com/show_bug.cgi?id=856501
https://bugzilla.novell.com/show_bug.cgi?id=856501#c
zhang jiajun
https://bugzilla.novell.com/show_bug.cgi?id=856501
https://bugzilla.novell.com/show_bug.cgi?id=856501#c2
--- Comment #2 from Rihards Olups
https://bugzilla.novell.com/show_bug.cgi?id=856501
https://bugzilla.novell.com/show_bug.cgi?id=856501#c3
David Haller
https://bugzilla.novell.com/show_bug.cgi?id=856501
https://bugzilla.novell.com/show_bug.cgi?id=856501#c4
--- Comment #4 from Bernhard Wiedemann
http://bugzilla.novell.com/show_bug.cgi?id=856501
Swamp Workflow Management
http://bugzilla.novell.com/show_bug.cgi?id=856501
--- Comment #5 from Swamp Workflow Management
http://bugzilla.novell.com/show_bug.cgi?id=856501
Srinidhi B S
http://bugzilla.novell.com/show_bug.cgi?id=856501
--- Comment #7 from David Haller
problems is a bit extreme, IMO. Fish is still useful for other actions. Instead of disabling file "move" functionality, we've disabled entire fish!
"Just" disabling "move" is not as easy as you might think. I did have a look at the code and just gave up. Have you ever looked at mc's "fish" code? I'd welcome any patches! -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=856501
--- Comment #8 from Srinidhi B S
(In reply to Srinidhi B from comment #6) Disabling SSH / fish completely because one action of "move" can cause
problems is a bit extreme, IMO. Fish is still useful for other actions. Instead of disabling file "move" functionality, we've disabled entire fish!
"Just" disabling "move" is not as easy as you might think. I did have a look at the code and just gave up.
I do understand that it isn't easy and may not be the Right Thing To Do. I didn't mean to suggest that it should be done this way - I just wanted to convey that some other less drastic options could have been looked at rather than completely removing fish / SSH functionality. Please understand that I've been using mc for almost 11 yrs now and have relied upon SSH/FTP functionality extensively. When I suddenly found this missing in openSUSE 13.2 - especially, when I needed it very badly - it annoyed me a little. Unfortunately, some of that annoyance came out in my previous comment. My apologies!! I really appreciate all your efforts in maintaining my favorite file manager in openSUSE.
Have you ever looked at mc's "fish" code? I'd welcome any patches!
I was expecting this response and I was planning to look at it. But I'm not confident if I'll be able to come up with any decent patch(es)! :) But today, I tried to reproduce this issue on my local build of mc where I've enabled fish using the steps mentioned in upstream bug / ticket, but wasn't able to reproduce the issue. Looks like "mc-vfs-fish-deleted_source_file.patch" does prevent the deletion of the source file. But one side-effect of this patch is that you get disconnected from remote SSH server! -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.novell.com/show_bug.cgi?id=856501
Steffen Sledz
http://bugzilla.novell.com/show_bug.cgi?id=856501
Swamp Workflow Management
http://bugzilla.novell.com/show_bug.cgi?id=856501
David Haller
I didn't mean to suggest that it should be done this way - I just wanted to convey that some other less drastic options could have been looked at rather than completely removing fish / SSH functionality.
That's what I did. But after hours, I gave up. And sftp is working just fine.
But today, I tried to reproduce this issue on my local build of mc where I've enabled fish using the steps mentioned in upstream bug / ticket, but wasn't able to reproduce the issue. Looks like "mc-vfs-fish-deleted_source_file.patch" does prevent the deletion of the source file. But one side-effect of this patch is that you get disconnected from remote SSH server!
Hm. That's good to hear. Well, you have a head start with that patch, pointing you to where stuff is done, I for one think that fish is broken by design and am not willing to spend much more time on that code. But I will spend time assessing and testing patches disabling only or fixing the "move" action. In that case, I'd be glad if you reopen this bug. Else, go bugger upstream ;) -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply@novell.com