Bug 2720 - Tapping enter on previous sessions in druid causes nullpointer
Summary: Tapping enter on previous sessions in druid causes nullpointer
Status: RESOLVED FIXED
Alias: None
Product: frysk
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
: P2 normal
Target Milestone: ---
Assignee: Sami Wagiaalla
URL:
Keywords:
Depends on:
Blocks: 3401
  Show dependency tree / graph
 
Reported: 2006-05-31 18:06 UTC by Mike Cvet
Modified: 2006-11-21 21:57 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:
Last reconfirmed:

Attachments
Fix (802 bytes, patch)
2006-06-01 16:09 UTC, Mike Cvet 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mike Cvet 2006-05-31 18:06:45 UTC 
If there multiple (i.e. 2) previous sessions in the druid, selecting one and
tapping enter twice renames it to itself, and selects the next item. Hitting
enter twice more causes a nullpointer:

java.lang.NullPointerException
   at frysk.gui.sessions.SessionManager.removeSession(FryskGui)
   at frysk.gui.druid.CreateFryskSessionDruid$1.cellRendererTextEvent(FryskGui)
   at org.gnu.gtk.CellRendererText.fireCellRendererTextEvent(libgtkjava-2.8.so)
   at org.gnu.gtk.CellRendererText.handleEdited(libgtkjava-2.8.so)
   at org.gnu.gtk.Gtk.gtk_main(libgtkjava-2.8.so)
   at org.gnu.gtk.Gtk.main(libgtkjava-2.8.so)
   at frysk.gui.Gui.gui(FryskGui)
   at frysk.gui.FryskGui.main(FryskGui)
Comment 1 Mike Cvet 2006-06-01 16:09:37 UTC 
Created attachment 1060 [details]
Fix

Prevents nulls from getting processed by CreateFryskSessionDruid, as well as
preventing empty strings getting assigned as new session names. Also added a
check for selections in ListView which could potentially throw a nullpointer if
there are no selected items.
Comment 2 Mike Cvet 2006-06-06 15:54:01 UTC 
Fixed in CVS.