changeset 7990:2efa310226f7

8023717: (process) ProcessBuilder should catch SecurityException rather than AccessControlException Reviewed-by: wetmore, alanb Contributed-by: martinrb@google.com
author alanb
date Wed, 28 Aug 2013 14:07:30 +0100
parents be2d25a277a7
children 378acd4d03c8 0aba8b6232af
files src/share/classes/java/lang/ProcessBuilder.java
diffstat 1 files changed, 2 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/classes/java/lang/ProcessBuilder.java	Wed Aug 21 20:41:35 2013 -0700
+++ b/src/share/classes/java/lang/ProcessBuilder.java	Wed Aug 28 14:07:30 2013 +0100
@@ -29,7 +29,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
-import java.security.AccessControlException;
 import java.util.Arrays;
 import java.util.ArrayList;
 import java.util.List;
@@ -1033,9 +1032,9 @@
                 // Can not disclose the fail reason for read-protected files.
                 try {
                     security.checkRead(prog);
-                } catch (AccessControlException ace) {
+                } catch (SecurityException se) {
                     exceptionInfo = "";
-                    cause = ace;
+                    cause = se;
                 }
             }
             // It's much easier for us to create a high-quality error