Mercurial > hg > release > icedtea7-forest-2.2 > jdk

changeset 5268:a487fa826258

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
7130662: GTK file dialog crashes with a NPE Summary: Guard adding a back slash to the directory name with an if (!= null) check Reviewed-by: anthony, art Contributed-by: Matt <melkor@orangepalantir.org>
author anthony
date Wed, 18 Jan 2012 19:09:26 +0400
parents 7e8272d637c7
children a981b386d095
files src/solaris/classes/sun/awt/X11/GtkFileDialogPeer.java
diffstat 1 files changed, 6 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/src/solaris/classes/sun/awt/X11/GtkFileDialogPeer.java	Mon Jan 28 15:15:10 2013 -0800
+++ b/src/solaris/classes/sun/awt/X11/GtkFileDialogPeer.java	Wed Jan 18 19:09:26 2012 +0400
@@ -76,9 +76,12 @@
             accessor.setFiles(fd, null);
         } else {
             // Fix 6987233: add the trailing slash if it's absent
-            accessor.setDirectory(fd, directory +
-                    (directory.endsWith(File.separator) ?
-                     "" : File.separator));
+            String with_separator = directory;
+            if (directory != null) {
+                with_separator = directory.endsWith(File.separator) ?
+                        directory : (directory + File.separator);
+            }
+            accessor.setDirectory(fd, with_separator);
             accessor.setFile(fd, filenames[0]);
 
             int filesNumber = (filenames != null) ? filenames.length : 0;