Looks good.
Mandy
On 7/18/2014 2:42 AM, Miroslav Kos wrote:
Good catch Mandy, thanks!
updated webrev: http://cr.openjdk.java.net/~mkos/8038970/jaxws.06/
JBS: https://bugs.openjdk.java.net/browse/JDK-8038970
Added JAF files (overwritten during integration from upstream),
removed throws/catch(es) in XMLStreamWriterFactory.
Thanks
Miran
On 17/07/14 18:02, Mandy Chung wrote:
On 7/17/2014 8:12 AM, Miroslav Kos wrote:
Hi everybody,
here is a request for review for
8038970: Deprivilege JAX-WS/JAF code
JBS: https://bugs.openjdk.java.net/browse/JDK-8038970
webrev: http://cr.openjdk.java.net/~mkos/8038970/jaxws.04/
FYI. The href at the above links were wrong.
com/sun/xml/internal/ws/api/streaming/XMLStreamWriterFactory.java
line 309: it's fine wrapping with NoSuchMethodError and
you can take out throws NoSuchMethodException from line 292
Did you miss JAF classes?
As a background, this patch is to prepare for granting minimal set
of permissions.
Mandy
This is first part of changes - after this also jdk build will have
to be changed.
Changeset has been already discussed during testing phase with Sean
Mullan and Mandy Chung.
Thanks
Miran