Bug 3619 - NPE's when source window is started with johntheripper
Summary: NPE's when source window is started with johntheripper
Status: RESOLVED FIXED
Alias: None
Product: frysk
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
: P2 normal
Target Milestone: ---
Assignee: Mike Cvet
URL:
Keywords:
Depends on:
Blocks: 1633
  Show dependency tree / graph
 
Reported: 2006-11-29 18:06 UTC by Rick Moseley
Modified: 2006-12-05 19:27 UTC (History)
1 user (show)

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

Attachments
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 Rick Moseley 2006-11-29 18:06:22 UTC 
I brought the johntheripper code up and tried to use the source window to attach
to it and got this error:

java.lang.NullPointerException
   at
frysk.gui.srcwin.SourceWindow$SourceWindowListener.currentStackChanged(FryskGui)
   at frysk.gui.srcwin.CurrentStackView.notifyObservers(FryskGui)
   at frysk.gui.srcwin.CurrentStackView.selectionChangedEvent(FryskGui)
   at org.gnu.gtk.TreeSelection.fireSelectionEvent(libgtkjava-2.8.so)
   at org.gnu.gtk.TreeSelection.handleChanged(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)

It also says "No debug information available for this stack frame" in the source
window when it comes up even though I have compiled it with the -g option.
Comment 1 Rick Moseley 2006-11-30 16:53:54 UTC 
This NPE appears when I click on the "#1 0x806c516 in DES_bs_crypt_LM()" in the
"Current Stack" view.
Comment 2 Mike Cvet 2006-12-05 19:06:38 UTC 
2006-12-05  Mike Cvet  <mcvet@redhat.com>

	* SourceBuffer.java (setCurrentLine): Check for null frame data.
	* SourceWindow.java: Removed references to SourceWindow state.
	(SourceWindowListener.currentStackChanged): Prevent nulls, fixing #3619.
	* SourceView.java (SourceViewListener.mouseMotionEvent): Remove
	reference to SourceWindow state macro.
Comment 3 Rick Moseley 2006-12-05 19:27:13 UTC 
Appears to be fixed in latest CVS head.