1 /*
   2  * Copyright (c) 2009, 2017, Oracle and/or its affiliates. All rights reserved.
   3  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   4  *
   5  * This code is free software; you can redistribute it and/or modify it
   6  * under the terms of the GNU General Public License version 2 only, as
   7  * published by the Free Software Foundation.
   8  *
   9  * This code is distributed in the hope that it will be useful, but WITHOUT
  10  * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
  11  * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
  12  * version 2 for more details (a copy is included in the LICENSE file that
  13  * accompanied this code).
  14  *
  15  * You should have received a copy of the GNU General Public License version
  16  * 2 along with this work; if not, write to the Free Software Foundation,
  17  * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
  18  *
  19  * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
  20  * or visit www.oracle.com if you need additional information or have any
  21  * questions.
  22  */
  23 
  24 /**
  25  * @test
  26  * @key headful
  27  * @bug 6713352
  28  * @summary Deadlock in JFileChooser with synchronized custom FileSystemView
  29  * @author Pavel Porvatov
  30  * @modules java.desktop/sun.awt.shell
  31  * @run main bug6713352
  32  */
  33 
  34 import sun.awt.shell.ShellFolder;
  35 
  36 import javax.swing.*;
  37 import javax.swing.filechooser.FileSystemView;
  38 import java.io.File;
  39 import java.io.FileNotFoundException;
  40 import java.io.IOException;
  41 
  42 public class bug6713352 {
  43     public static void main(String[] args) throws Exception {
  44         SwingUtilities.invokeAndWait(new Runnable() {
  45             public void run() {
  46                 String tempDir = System.getProperty("java.io.tmpdir");
  47 
  48                 if (tempDir == null || !new File(tempDir).isDirectory()) {
  49                     tempDir = System.getProperty("user.home");
  50                 }
  51 
  52                 MyFileSystemView systemView = new MyFileSystemView();
  53 
  54                 synchronized (systemView) { // Get SystemView lock
  55                     new JFileChooser(systemView);
  56 
  57                     // Wait a little bit. BasicDirectoryModel will lock Invoker and stop on
  58                     // the bug6713352.MyFileSystemView.getFiles() method
  59                     try {
  60                         Thread.sleep(5000);
  61                     } catch (InterruptedException e) {
  62                         throw new RuntimeException(e);
  63                     }
  64 
  65                     try {
  66                         System.out.println("Try to get Invokers lock");
  67 
  68                         ShellFolder.getShellFolder(new File(tempDir)).listFiles(true);
  69                     } catch (FileNotFoundException e) {
  70                         throw new RuntimeException(e);
  71                     }
  72                 }
  73 
  74                 // To avoid RejectedExecutionException in BasicDirectoryModel wait a second
  75                 try {
  76                     Thread.sleep(1000);
  77                 } catch (InterruptedException e) {
  78                     throw new RuntimeException(e);
  79                 }
  80             }
  81         });
  82     }
  83 
  84     private static class MyFileSystemView extends FileSystemView {
  85 
  86         public File createNewFolder(File containingDir) throws IOException {
  87             return null;
  88         }
  89 
  90         public File[] getFiles(File dir, boolean useFileHiding) {
  91             System.out.println("getFiles start");
  92 
  93             File[] result;
  94 
  95             synchronized (this) {
  96                 result = super.getFiles(dir, useFileHiding);
  97             }
  98 
  99             System.out.println("getFiles finished");
 100 
 101             return result;
 102         }
 103 
 104         public synchronized Boolean isTraversable(File f) {
 105             return super.isTraversable(f);
 106         }
 107     }
 108 }