New issue
Advanced search Search tips

Issue 788285 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

nasftp: rename fails when a mount path is set

Project Member Reported by reillyg@chromium.org, Nov 24 2017

Issue description

Chrome Version: 63.0.3239.50
OS: Chrome OS 10032.39.0

What steps will reproduce the problem?
(1) Use the Secure Shell app to mount an SFTP server and specify a mount path.
(2) Create a directory on the remote server.
(3) Attempt to rename this directory from "New Folder".

What is the expected result?
Directory should be renamed.

What happens instead?
An error is reported.
 

Comment 1 by vapier@chromium.org, Nov 24 2017

Summary: nasftp: rename fails when a mount path is set (was: [Secure Shell SFTP] Rename fails when a mount path is set)
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/apps/libapps/+/b73450642cdac98a0125825020c26a14a8a5cd3c

commit b73450642cdac98a0125825020c26a14a8a5cd3c
Author: Reilly Grant <reillyg@chromium.org>
Date: Fri Nov 24 15:09:46 2017

nassh: Set base path for SFTP RENAME and SETSTAT packets

For these packets the path was not being prefixed with the base mount
path which meant that the server would report "no such file" when not
using the default mount path and performing a rename.

Bug:  chromium:788285 
Change-Id: I84d46a10597f66d61ec8ba684d6a0e545fcf50c2
Reviewed-on: https://chromium-review.googlesource.com/788730
Tested-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/b73450642cdac98a0125825020c26a14a8a5cd3c/nassh/js/nassh_sftp_client.js

Status: Fixed (was: Started)

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 5 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment