Commitd47025c

committed

lib/Horde/Vfs/Ssh2.php: ssh2_sftp_statinfo() returns boolean false on non-existent files/folders. Avoid accessing those return results as arrays.

1 parent0613ccb commitd47025cCopy full SHA for d47025c

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,7 @@ public function size($path, $name)`
`101`	`101`	`$this->_connect();`
`102`	`102`
`103`	`103`	`$statinfo = @ssh2_sftp_stat($this->_sftp,$this->_getPath($path,$name));`
`104`		`-if (($size =$statinfo['size']) ===false) {`
	`104`	`+if ( ($statinfo ===false)or (($size =$statinfo['size']) ===false)) {`
`105`	`105`	`thrownewHorde_Vfs_Exception(sprintf('Unable to check file size of "%s".',$this->_getPath($path,$name)));`
`106`	`106`	`}`
`107`	`107`
`@@ -260,7 +260,7 @@ public function isFolder($path, $name)`
`260`	`260`	`/* See if we can stat the remote filename. ANDed with 040000 is true`
`261`	`261`	`* if it is a directory. */`
`262`	`262`	`$statinfo = @ssh2_sftp_stat($this->_sftp,$this->_getPath($path,$name));`
`263`		`-return$statinfo['mode'] &040000;`
	`263`	`+return($statinfo !==false )and ($statinfo['mode'] &040000 );`
`264`	`264`	`}`
`265`	`265`
`266`	`266`	`/**`

Comments

(0)