[VFS-258] [SFTP][RAM] Unsafe casting to AbstractFileObject subclasses in doRename().

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0
• Fix Version/s: 2.1
• Assignee: Unassigned
• Reporter: Marek Zawirski
• Created: 19/May/09 13:11
• Updated: 19/May/16 18:25
• Resolved: 16/May/12 15:06

Description

AbstractFileObject#doRename() method is called from AbstractFileObject#moveTo() when file can be moved within the same file system. As it concerns file that is subclass AbstractFileObject, target file is also assumed to be AbstractFileObject type. However, this target file can be decorated. Undressing with FileObjectUtils.getAbstractFileObject() was not performed in every places that it should be. Some subclasses do correct stripping of decorator in doRename() implementations (e.g. FtpFileObject), some of them not (e.g. RamFileObject) - which may cause ClassCastExceptions.

Patch proposal: pass undressed AbstractFileObject to doRename() instead of possibly decorated FileObject.