[Mono-bugs] [Bug 81315][Maj] New - TreeView: NRE caused by right-click with no selected node

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by monkey at jpobst.com.

http://bugzilla.ximian.com/show_bug.cgi?id=81315

--- shadow/81315	2007-04-05 00:05:11.000000000 -0400
+++ shadow/81315.tmp.28523	2007-04-05 00:05:11.000000000 -0400
@@ -0,0 +1,69 @@
+Bug#: 81315
+Product: Mono: Class Libraries
+Version: 1.2
+OS: 
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Major
+Component: Windows.Forms
+AssignedTo: toshok at ximian.com                            
+ReportedBy: monkey at jpobst.com               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: TreeView: NRE caused by right-click with no selected node
+
+Steps to reproduce the problem:
+1. Compile and run attached program
+2. Right click node (without left clicking first)
+
+Produces:
+Unhandled Exception: System.NullReferenceException: Object reference not
+set to an instance of an object
+
+at System.Windows.Forms.TreeView.MouseUpHandler
+(object,System.Windows.Forms.MouseEventArgs) [0x00047] in TreeView.cs:1840
+at (wrapper delegate-invoke)
+System.MulticastDelegate.invoke_void_object_MouseEventArgs
+(object,System.Windows.Forms.MouseEventArgs) <0x0003e>
+at System.Windows.Forms.Control.OnMouseUp
+(System.Windows.Forms.MouseEventArgs) [0x0001c] in Control.cs:5518
+at System.Windows.Forms.Control.WmRButtonUp (System.Windows.Forms.Message&)
+[0x000c3] in Control.cs:4904
+at System.Windows.Forms.Control.WndProc (System.Windows.Forms.Message&)
+[0x00196] in Control.cs:4627
+at System.Windows.Forms.TreeView.WndProc (System.Windows.Forms.Message&)
+[0x00051] in TreeView.cs:989
+at ControlWindowTarget.OnMessage (System.Windows.Forms.Message&) [0x00000]
+in Control.cs:222
+at ControlNativeWindow.WndProc (System.Windows.Forms.Message&) [0x00000] in
+Control.cs:203
+at System.Windows.Forms.NativeWindow.WndProc
+(intptr,System.Windows.Forms.Msg,intptr,intptr) [0x0006b] in
+NativeWindow.cs:179
+at System.Windows.Forms.XplatUIWin32.InternalWndProc
+(intptr,System.Windows.Forms.Msg,intptr,intptr) [0x0002d] in
+XplatUIWin32.cs:1562
+at (wrapper native-to-managed)
+System.Windows.Forms.XplatUIWin32.InternalWndProc
+(intptr,System.Windows.Forms.Msg,intptr,intptr) <0x00051>
+in (unmanaged) 77281A0F
+at (wrapper managed-to-native)
+System.Windows.Forms.XplatUIWin32.Win32DispatchMessage
+(System.Windows.Forms.MSG&) <0x00004>
+at System.Windows.Forms.XplatUIWin32.DispatchMessage
+(System.Windows.Forms.MSG&) [0x00000] in XplatUIWin32.cs:1757
+at System.Windows.Forms.XplatUI.DispatchMessage (System.Windows.Forms.MSG&)
+[0x00000] in XplatUI.cs:437
+at System.Windows.Forms.Application.RunLoop
+(bool,System.Windows.Forms.ApplicationContext) [0x00318] in Application.cs:645
+at System.Windows.Forms.Application.Run (System.Windows.Forms.Form)
+[0x00000] in Application.cs:499
+at MainForm.Main (string[]) <0x0001f>
+
+which is this line:
+Invalidate (selected_node.Bounds);
+because the selected_node is null.