stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8023717: (process) ProcessBuilder should catch SecurityException rather than AccessControlException

Browse Source 
Reviewed-by: wetmore, alanb
This commit is contained in:
Martin Buchholz 2013-08-28 14:07:30 +01:00 committed by Alan Bateman
parent 1d19a4f5df
commit ce3943d27d
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
jdk/src/share/classes/java/lang/ProcessBuilder.java
View File

@ -29,7 +29,6 @@ import java.io.File;
import java.io.IOException; import java.io.IOException;
import java.io.InputStream; import java.io.InputStream;
import java.io.OutputStream; import java.io.OutputStream;
import java.security.AccessControlException;
import java.util.Arrays; import java.util.Arrays;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
@ -1033,9 +1032,9 @@ public final class ProcessBuilder
// Can not disclose the fail reason for read-protected files. // Can not disclose the fail reason for read-protected files.
try { try {
security.checkRead(prog); security.checkRead(prog);
} catch (AccessControlException ace) { } catch (SecurityException se) {
exceptionInfo = ""; exceptionInfo = "";
cause = ace; cause = se;
} }
} }
// It's much easier for us to create a high-quality error // It's much easier for us to create a high-quality error