[Kst] [Bug 128374] Crash when trying to modify a legend

George Staikos staikos at kde.org
Wed May 31 15:12:46 CEST 2006


------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=128374         
staikos kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From staikos kde org  2006-05-31 15:12 -------
SVN commit 546935 by staikos:

Ok now I see how this can happen.  Don't allow recursive menus at all.  It's too
dangerous.
BUG: 128374


 M  +4 -1      kstviewwidget.cpp  


--- trunk/extragear/graphics/kst/src/libkstapp/kstviewwidget.cpp #546934:546935
 @ -324,7 +324,10  @
   if (_view->mouseGrabber()) {
     _view->releaseMouse(_view->mouseGrabber());
   }
-  assert(!_menu);
+  if (_menu) { // Do not allow recursive menus
+    e->ignore();
+    return;
+  }
   _menu = new KPopupMenu(this);
   //kstdDebug() << "Querying for the popup" << endl;
   bool rc = _view->popupMenu(_menu, e->pos());


More information about the Kst mailing list