1 /*
   2  * Copyright (c) 2009, 2015, 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 /* @test
  25    @bug 6713352
  26    @summary Deadlock in JFileChooser with synchronized custom FileSystemView
  27    @author Pavel Porvatov
  28    @modules java.desktop/sun.awt.shell
  29    @run main bug6713352
  30 */
  31 
  32 import sun.awt.shell.ShellFolder;
  33 
  34 import javax.swing.*;
  35 import javax.swing.filechooser.FileSystemView;
  36 import java.io.File;
  37 import java.io.FileNotFoundException;
  38 import java.io.IOException;
  39 
  40 public class bug6713352 {
  41     public static void main(String[] args) throws Exception {
  42         SwingUtilities.invokeAndWait(new Runnable() {
  43             public void run() {
  44                 String tempDir = System.getProperty("java.io.tmpdir");
  45 
  46                 if (tempDir == null || !new File(tempDir).isDirectory()) {
  47                     tempDir = System.getProperty("user.home");
  48                 }
  49 
  50                 MyFileSystemView systemView = new MyFileSystemView();
  51 
  52                 synchronized (systemView) { // Get SystemView lock
  53                     new JFileChooser(systemView);
  54 
  55                     // Wait a little bit. BasicDirectoryModel will lock Invoker and stop on
  56                     // the bug6713352.MyFileSystemView.getFiles() method
  57                     try {
  58                         Thread.sleep(5000);
  59                     } catch (InterruptedException e) {
  60                         throw new RuntimeException(e);
  61                     }
  62 
  63                     try {
  64                         System.out.println("Try to get Invokers lock");
  65 
  66                         ShellFolder.getShellFolder(new File(tempDir)).listFiles(true);
  67                     } catch (FileNotFoundException e) {
  68                         throw new RuntimeException(e);
  69                     }
  70                 }
  71 
  72                 // To avoid RejectedExecutionException in BasicDirectoryModel wait a second
  73                 try {
  74                     Thread.sleep(1000);
  75                 } catch (InterruptedException e) {
  76                     throw new RuntimeException(e);
  77                 }
  78             }
  79         });
  80     }
  81 
  82     private static class MyFileSystemView extends FileSystemView {
  83 
  84         public File createNewFolder(File containingDir) throws IOException {
  85             return null;
  86         }
  87 
  88         public File[] getFiles(File dir, boolean useFileHiding) {
  89             System.out.println("getFiles start");
  90 
  91             File[] result;
  92 
  93             synchronized (this) {
  94                 result = super.getFiles(dir, useFileHiding);
  95             }
  96 
  97             System.out.println("getFiles finished");
  98 
  99             return result;
 100         }
 101 
 102         public synchronized Boolean isTraversable(File f) {
 103             return super.isTraversable(f);
 104         }
 105     }
 106 }