Applied and committed. Thanks. In message <[EMAIL PROTECTED]> on Tue, 03 Jun 2003 10:53:22 +0200, Claude Lecommandeur <[EMAIL PROTECTED]> said:
claude.lecommandeur> claude.lecommandeur> claude.lecommandeur> -------- Original Message -------- claude.lecommandeur> Subject: memory leak patch for ctwm claude.lecommandeur> Date: Tue, 3 Jun 2003 10:09:40 +1000 claude.lecommandeur> From: Simon Burge <[EMAIL PROTECTED]> claude.lecommandeur> Organization: Wasabi Systems, Inc. claude.lecommandeur> To: Claude Lecommandeur <[EMAIL PROTECTED]> claude.lecommandeur> claude.lecommandeur> claude.lecommandeur> claude.lecommandeur> Hi Claude, claude.lecommandeur> claude.lecommandeur> The following patch fixes a memory leak. My ~2 week old ctwm was claude.lecommandeur> up to about 36MB in size. The text_prop.value is malloced on calls claude.lecommandeur> to XGetTextProperty, but nothing frees that memory in the call in claude.lecommandeur> util.c:GetWMPropertyString(). claude.lecommandeur> claude.lecommandeur> Note that I really don't know the first thing about X11 programming claude.lecommandeur> (I'm a kernel person), but it looks like it's ok to free what I'm claude.lecommandeur> freeing where I've done it based on other calls to XGetTextProperty() claude.lecommandeur> I could see. -- Richard Levitte | http://richard.levitte.org/ | Tunnlandsv. 3 Levitte Programming | http://www.lp.se/ | S-168 36 Bromma T: +46-708-26 53 44 | | SWEDEN "Price, performance, quality... choose the two you like"
