[patch] PickProcDialog vs Proc.getExe()

[Mark Wielaard <mark@klomp.org>]

Hi,

On Wed, 2007-07-25 at 13:53 +0200, Mark Wielaard wrote:
> This break FryskGui which isn't prepared to handle null from Proc.get().

Here is a workaround for at least one case (there might be others, but I
haven't looked too deeply yet). This gets FryskGui up and running again
for me.

2007-07-25  Mark Wielaard  <mwielaard@redhat.com>

    * PickProcDialog.java (ProcCreatedObserver.run): Check whether
    Proc.getExe() returns null before constructing File path.

Cheers,

Mark

diff -u -r1.12 PickProcDialog.java
--- frysk-gui/frysk/gui/monitor/PickProcDialog.java     30 Jan 2007 00:18:21 -0000      1.12
+++ frysk-gui/frysk/gui/monitor/PickProcDialog.java     25 Jul 2007 11:55:53 -0000
@@ -355,11 +355,14 @@
             {
               iterMap.put(proc.getCommand(), process);
               treeStore.setValue(process, nameDC, proc.getCommand());
-              File path = new File(proc.getExe());
-              
-              if (path != null)
-                treeStore.setValue(process, locationDC,
-                                   justPath(path.getPath(), path.getName()));
+
+             String exe = proc.getExe();
+             if (exe != null)
+               {
+                 File path = new File(exe);
+                 treeStore.setValue(process, locationDC,
+                                    justPath(path.getPath(), path.getName()));
+               }
               else
                 treeStore.setValue(process, locationDC, "");