You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@felix.apache.org by gn...@apache.org on 2016/03/21 17:59:28 UTC

svn commit: r1736053 - /felix/trunk/gogo/runtime/src/main/java/org/apache/felix/gogo/runtime/Pipe.java

Author: gnodet
Date: Mon Mar 21 16:59:28 2016
New Revision: 1736053

URL: http://svn.apache.org/viewvc?rev=1736053&view=rev
Log:
Avoid possible exceptions with isTty

Modified:
    felix/trunk/gogo/runtime/src/main/java/org/apache/felix/gogo/runtime/Pipe.java

Modified: felix/trunk/gogo/runtime/src/main/java/org/apache/felix/gogo/runtime/Pipe.java
URL: http://svn.apache.org/viewvc/felix/trunk/gogo/runtime/src/main/java/org/apache/felix/gogo/runtime/Pipe.java?rev=1736053&r1=1736052&r2=1736053&view=diff
==============================================================================
--- felix/trunk/gogo/runtime/src/main/java/org/apache/felix/gogo/runtime/Pipe.java (original)
+++ felix/trunk/gogo/runtime/src/main/java/org/apache/felix/gogo/runtime/Pipe.java Mon Mar 21 16:59:28 2016
@@ -203,7 +203,10 @@ public class Pipe implements Callable<Re
 
     public boolean isTty(int fd) {
         // TODO: this assumes that the session is always created with input/output tty streams
-        return !toclose[fd];
+        if (fd < 0 || fd > streams.length) {
+            return false;
+        }
+        return streams[fd] != null && !toclose[fd];
     }
 
     public void error(int error) {