[
https://issues.apache.org/jira/browse/VFS-286?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kirill Safonov updated VFS-286:
-------------------------------
Description:
SftpFileObject.doListChildrenResolved() changes the working dir before doing
ChannelSftp.ls() call. If ls() throws an exception (in my case it's
SftpException "No such file" when trying to list the contents of "/dev/cdrom"),
the execution is blown out the method and channel.cd(workingDirectory) is not
performed. *All* the subsequent operations that rely on the current dir will
fail trying to cd() into unexisting directory.
The fix will be to move the channel.cd(workingDirectory); part into the finally
{} block below
was:
SftpFileObject.doListChildrenResolved() changes the working dir before doing
ChannelSftp.ls() call. If ls() throws an exception (in my case it's
SftpException "No such file" when trying to list the contents of "/dev/cdrom"),
the execution is blown out the method and channel.cd(workingDirectory) is not
performed. *All* the subsequent operations that rely on the current dir will
fail trying to cd() into unexisting directory.
This fix will be to move the channel.cd(workingDirectory); part into the
finally {} block below
> SFTP: getChildren() does not restore working dir if exception is thrown
> -----------------------------------------------------------------------
>
> Key: VFS-286
> URL: https://issues.apache.org/jira/browse/VFS-286
> Project: Commons VFS
> Issue Type: Bug
> Affects Versions: Nightly Builds
> Reporter: Kirill Safonov
> Priority: Critical
>
> SftpFileObject.doListChildrenResolved() changes the working dir before doing
> ChannelSftp.ls() call. If ls() throws an exception (in my case it's
> SftpException "No such file" when trying to list the contents of
> "/dev/cdrom"), the execution is blown out the method and
> channel.cd(workingDirectory) is not performed. *All* the subsequent
> operations that rely on the current dir will fail trying to cd() into
> unexisting directory.
> The fix will be to move the channel.cd(workingDirectory); part into the
> finally {} block below
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
