diff options
| author | Lukas Reschke <lukas@owncloud.com> | 2015-06-10 14:46:42 +0200 |
|---|---|---|
| committer | Lukas Reschke <lukas@owncloud.com> | 2015-06-10 14:46:42 +0200 |
| commit | 14c760124fd0eb92684297c587769cea1d11f2f2 (patch) | |
| tree | 568bc344ecd7acf0b7dd68dd70953b06faada92a /apps | |
| parent | c391f74c2e80e6237526631020c775dd87d33010 (diff) | |
| download | nextcloud-server-14c760124fd0eb92684297c587769cea1d11f2f2.tar.gz nextcloud-server-14c760124fd0eb92684297c587769cea1d11f2f2.zip | |
Don't use command concatenation
Possibly fixes https://github.com/owncloud/core/issues/16853
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/files_external/3rdparty/smb4php/smb.php | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/apps/files_external/3rdparty/smb4php/smb.php b/apps/files_external/3rdparty/smb4php/smb.php index 7ffdb42e134..5db663b0bde 100644 --- a/apps/files_external/3rdparty/smb4php/smb.php +++ b/apps/files_external/3rdparty/smb4php/smb.php @@ -324,14 +324,14 @@ class smb { trigger_error('rename(): error in URL', E_USER_ERROR); } smb::clearstatcache ($url_from); - $cmd = ''; // check if target file exists if (smb::url_stat($url_to)) { // delete target file first - $cmd = 'del "' . $to['path'] . '"; '; + $cmd = 'del "' . $to['path'] . '"'; + smb::execute($cmd, $to); $replace = true; } - $cmd .= 'rename "' . $from['path'] . '" "' . $to['path'] . '"'; + $cmd = 'rename "' . $from['path'] . '" "' . $to['path'] . '"'; $result = smb::execute($cmd, $to); if ($replace) { // clear again, else the cache will return the info |