<Swing Dev> [10] RFR 6962725:Regtest javax/swing/JFileChooser/6738668/bug6738668.java fails under Linux

[Sergey Bylokhov]

The new version fails even if run via jtreg?

----- prasanta.sadhukhan at oracle.com wrote:

> OK. With revised command line, it fails with new updated test in 7b40
> 
> and passed in 9b170
> 
> /cygdrive/d/Vbox_Shared/JDK/jdk1.7.0-b40/fastdebug/bin/java 
> -Djava.security.manager -Djava.security.policy=security.policy
> bug6738668
> tmp dir C:\cygwin64\tmp\
> Exception in thread "main" java.security.AccessControlException:
> access 
> denied (java.io.FilePermission 
> C:\Users\prsadhuk\AppData\Roaming\Microsoft\Windows\Recent read)
>          at 
> java.security.AccessControlContext.checkPermission(AccessControlContext.java:345)
>          at 
> java.security.AccessController.checkPermission(AccessController.java:555)
>          at 
> java.lang.SecurityManager.checkPermission(SecurityManager.java:549)
>          at
> java.lang.SecurityManager.checkRead(SecurityManager.java:888)
> 
> /cygdrive/d/Vbox_Shared/JDK/jdk9-b170/fastdebug/bin/java 
> -Djava.security.manager -Djava.security.policy=security.policy
> bug6738668
> tmp dir C:\cygwin64\tmp\
> Test passed for LookAndFeel javax.swing.plaf.metal.MetalLookAndFeel
> tmp dir C:\cygwin64\tmp\
> Test passed for LookAndFeel javax.swing.plaf.nimbus.NimbusLookAndFeel
> tmp dir C:\cygwin64\tmp\
> Test passed for LookAndFeel
> com.sun.java.swing.plaf.motif.MotifLookAndFeel
> tmp dir C:\cygwin64\tmp\
> Test passed for LookAndFeel 
> com.sun.java.swing.plaf.windows.WindowsLookAndFeel
> tmp dir C:\cygwin64\tmp\
> Test passed for LookAndFeel 
> com.sun.java.swing.plaf.windows.WindowsClassicLookAndFeel
> 
> Regards
> Prasanta
> On 5/30/2017 2:59 AM, Sergey Bylokhov wrote:
> > I guess you will need to set security manager which will use your
> policy file:
> >
> https://docs.oracle.com/cd/E13222_01/wls/docs81b/secmanage/java.html
> >
> > ----- prasanta.sadhukhan at oracle.com wrote:
> >
> >> I am not sure why it is passing in my case. I thought it might be
> >> taking
> >> in some other security policy so I renamed and use a different
> named
> >> policy but it still passed in my windows 7.
> >>
> >> $ /cygdrive/d/Vbox_Shared/JDK/jdk1.7.0-b40/fastdebug/bin/java
> >> -Djava.security.policy=my.policy bug6738668
> >> Test passed for LookAndFeel
> javax.swing.plaf.metal.MetalLookAndFeel
> >> Test passed for LookAndFeel
> >> com.sun.java.swing.plaf.motif.MotifLookAndFeel
> >> Test passed for LookAndFeel
> >> com.sun.java.swing.plaf.windows.WindowsLookAndFeel
> >> Test passed for LookAndFeel
> >> com.sun.java.swing.plaf.windows.WindowsClassicLookAndFeel
> >>
> >> PRSADHUK-IN+prsadhuk at PRSADHUK-IN
> >> /cygdrive/d/jdk10/client/jdk/test/javax/swing/JFileChooser/6738668
> >> $ cat my.policy
> >> grant {
> >>       permission java.io.FilePermission "C:\\temp\\*", "read";
> >>       permission java.io.FilePermission "C:\\temp", "read";
> >>       permission java.util.PropertyPermission "*", "read";
> >> };
> >>
> >> PRSADHUK-IN+prsadhuk at PRSADHUK-IN
> >> /cygdrive/d/jdk10/client/jdk/test/javax/swing/JFileChooser/6738668
> >> $ /cygdrive/d/Vbox_Shared/JDK/jdk1.7.0-b40/fastdebug/bin/java
> >> -version
> >> java version "1.7.0-ea-fastdebug"
> >> Java(TM) SE Runtime Environment (build 1.7.0-ea-fastdebug-b40)
> >> Java HotSpot(TM) 64-Bit Server VM (build 14.0-b07-fastdebug, mixed
> >> mode)
> >>
> >> BTW, does my updated test fail in your environment with b40?
> >>
> >> Regards
> >> Prasanta
> >> On 5/28/2017 4:10 AM, Sergey Bylokhov wrote:
> >>> There is a comment that JDK-6738668 a regression of JDK-6484091
> >> which was pushed to b20.
> >>> I just checked the test from the command line on b40 and it
> fails,
> >> but passed on b55.
> >>> Can you please check why it was passed in your case.
> >>>
> >>> ----- prasanta.sadhukhan at oracle.com wrote:
> >>>
> >>>> Hi Sergey,
> >>>>
> >>>> I do not see any comment mention about failure in 7b20 but
> anyways,
> >> I
> >>>> tried with 7b20,7b30,7b40,7b50 and all of them passed with
> >> original
> >>>> and
> >>>> updated test in windows.
> >>>>
> >>>> Regards
> >>>> Prasanta
> >>>> On 5/25/2017 2:30 AM, Sergey Bylokhov wrote:
> >>>>> I am not sure but according to the comments of JDK-6738668 it
> was
> >> a
> >>>> regression in 7b20.
> >>>>> So I suggest to check a few build between b20 and b54
> >>>>>
> >>>>> ----- prasanta.sadhukhan at oracle.com wrote:
> >>>>>
> >>>>>> Hi Sergey,
> >>>>>>
> >>>>>> No, it is passing. Actually, the original test is also passing
> >>>> with
> >>>>>> 1.7.0 b05 (6738668 is supposedly fixed in 7b55), there's no
> >>>>>> SecurityException
> >>>>>>
> >>>>>> jdk1.7.0/bin/java -Djava.security.policy=security.policy
> >>>> bug6738668
> >>>>>> Test passed for LookAndFeel
> >>>> javax.swing.plaf.metal.MetalLookAndFeel
> >>>>>> Test passed for LookAndFeel
> >>>>>> com.sun.java.swing.plaf.motif.MotifLookAndFeel
> >>>>>> Test passed for LookAndFeel
> >>>>>> com.sun.java.swing.plaf.windows.WindowsLookAndFeel
> >>>>>> Test passed for LookAndFeel
> >>>>>> com.sun.java.swing.plaf.windows.WindowsClassicLookAndFeel
> >>>>>>
> >>>>>> jdk1.7.0/bin/java -version
> >>>>>> java version "1.7.0-ea"
> >>>>>> Java(TM) SE Runtime Environment (build 1.7.0-ea-b05)
> >>>>>> Java HotSpot(TM) 64-Bit Server VM (build 1.7.0-ea-b05, mixed
> >> mode)
> >>>>>> Regards
> >>>>>> Prasanta
> >>>>>> On 5/24/2017 11:48 AM, Sergey Bylokhov wrote:
> >>>>>>> Hi, Prasanta.
> >>>>>>> Please confirm that the updated test fails before JDK-6738668
> >> was
> >>>>>> fixed.
> >>>>>>>> Hi ALl,
> >>>>>>>>
> >>>>>>>> Please review a testbug fix for an issue where this
> regression
> >>>> test
> >>>>>> is failing in linux because
> >>>>>>>> JFileChooser was trying to access C:/temp path which does
> not
> >>>> exist
> >>>>>> in linux.
> >>>>>>>> Proposed fix is to use java.io.tmpdir instead of hardcoded
> >>>> windows
> >>>>>> path.
> >>>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-6962725
> >>>>>>>> webrev:
> >>>> http://cr.openjdk.java.net/~psadhukhan/6962725/webrev.00/
> >>>>>>>> Regards
> >>>>>>>> Prasanta