Path: utzoo!utgpu!news-server.csri.toronto.edu!cs.utexas.edu!wuarchive!sdd.hp.com!elroy.jpl.nasa.gov!ncar!gatech!usenet.ins.cwru.edu!agate!ICSI.Berkeley.EDU!stolcke From: stolcke@ICSI.Berkeley.EDU (Andreas Stolcke) Newsgroups: comp.windows.x Subject: Re: tvtwm patch 5 dumps core Message-ID: <1991May11.004433.14527@agate.berkeley.edu> Date: 11 May 91 00:44:33 GMT References: Sender: root@agate.berkeley.edu (Charlie Root) Organization: International Computer Science Institute, Berkeley, CA Lines: 34 In article , jsparkes@bwdls49.bnr.ca (Jeff Sparkes) writes: |> |> It's dereferencing a null menu pointer. If you have menus on more |> than one button: hold one down, and then press a button that brings up |> another menu. Boom! That's gross! How about the following fix. -- Andreas Stolcke stolcke@icsi.berkeley.edu International Computer Science Institute stolcke@ucbicsi.bitnet 1957 Center St., Suite 600, Berkeley, CA 94704 (415) 642-4274 ext. 126 *** menus.c.dist Tue Apr 23 07:17:14 1991 --- menus.c Fri May 10 17:32:06 1991 *************** *** 569,576 **** } ! /* if we haven't recieved the enter notify yet, wait */ ! if (ActiveMenu && !ActiveMenu->entered) continue; done = FALSE; --- 569,576 ---- } ! /* if we haven't received the enter notify yet, wait */ ! if (!ActiveMenu || !ActiveMenu->entered) continue; done = FALSE;