You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mina.apache.org by gn...@apache.org on 2011/04/05 12:55:28 UTC

svn commit: r1088966 - /mina/sshd/trunk/sshd-core/src/main/java/org/apache/sshd/server/sftp/SftpSubsystem.java

Author: gnodet
Date: Tue Apr  5 10:55:28 2011
New Revision: 1088966

URL: http://svn.apache.org/viewvc?rev=1088966&view=rev
Log:
[SSHD-115] Missing return handling SFTP SSH_FXP_OPEN cannot create error

Modified:
    mina/sshd/trunk/sshd-core/src/main/java/org/apache/sshd/server/sftp/SftpSubsystem.java

Modified: mina/sshd/trunk/sshd-core/src/main/java/org/apache/sshd/server/sftp/SftpSubsystem.java
URL: http://svn.apache.org/viewvc/mina/sshd/trunk/sshd-core/src/main/java/org/apache/sshd/server/sftp/SftpSubsystem.java?rev=1088966&r1=1088965&r2=1088966&view=diff
==============================================================================
--- mina/sshd/trunk/sshd-core/src/main/java/org/apache/sshd/server/sftp/SftpSubsystem.java (original)
+++ mina/sshd/trunk/sshd-core/src/main/java/org/apache/sshd/server/sftp/SftpSubsystem.java Tue Apr  5 10:55:28 2011
@@ -409,6 +409,7 @@ public class SftpSubsystem implements Co
                             if (((pflags & SSH_FXF_CREAT) != 0)) {
                                 if (!file.isWritable()) {
                                     sendStatus(id, SSH_FX_FAILURE, "Can not create " + path);
+                                    return;
                                 }
                                 file.create();
                             }