Ticket #2582 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

Crash in 4.8.0-pre1 when copying symlink over ssh

Reported by: annulen Owned by: slavazanko
Priority: major Milestone: 4.8.0-pre2
Component: mc-vfs Version: 4.8.0-pre1
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: commited-master

Description

mc 4.8.0-pre1 crashes when copying symlink over ssh. Regular files are copied fine

Attachments

backtrace (4.6 KB) - added by annulen 5 years ago.
Backtrace from gdb

Change History

Changed 5 years ago by annulen

Backtrace from gdb

comment:1 Changed 5 years ago by slavazanko

  • Status changed from new to accepted
  • Owner set to slavazanko

comment:2 Changed 5 years ago by slavazanko

  • Keywords fish added
  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.0-pre2

Created branch 2582_sshfs_crash_while_copying_symlink

Initial changeset:bbe5ae99315daef50ca0e035b3338128236dcbc3

review, please.

comment:3 Changed 5 years ago by angel_il

  • Votes for changeset set to angel_il

comment:4 Changed 5 years ago by andrew_b

  • Keywords fish removed
  • Votes for changeset angel_il deleted
  • Branch state changed from on review to on rework

After copy to remote host, relative symbolic links become absolute and wrong. For example, if link foo points to bar/baz, after copy foo points to /bar/baz.

comment:5 Changed 5 years ago by slavazanko

  • Branch state changed from on rework to on review
Last edited 5 years ago by andrew_b (previous) (diff)

comment:6 Changed 5 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:7 Changed 5 years ago by angel_il

  • Votes for changeset changed from andrew_b to andrew_b angel_il
  • Branch state changed from on review to approved

comment:8 Changed 5 years ago by slavazanko

  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b angel_il to commited-master
  • Resolution set to fixed
  • Branch state changed from approved to merged

comment:9 Changed 5 years ago by slavazanko

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.