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


Reply via email to