Re: Re: fix crash in winecfg's x11 dialog due to invalid "Desktop" settings
Yes, my bad. I'll send in a fixed patch. I still don't like the way the code looks with the duplication of the default string to use but I couldn't think of any way to fix it more elegantly. Ideas? Chris
From: Mike Hearn <mh(a)codeweavers.com> Date: 2004/03/29 Mon AM 06:07:03 EST To: wine-patches(a)winehq.com Subject: Re: fix crash in winecfg's x11 dialog due to invalid "Desktop" settings
On Sun, 28 Mar 2004 13:31:15 -0500, Chris Morgan wrote:
+ if(!bufindex) /* handle invalid "Desktop" values */ + { + buf = strdup("640x480"); + bufindex = strchr(buf, 'x'); + }
It's not a big deal but there's a memory leak there, you need to free buf before strduping the default (which ideally should be a const str at the top of the function so if/when we change the default it's only done in one place).
thanks -mike
participants (1)
-
chmorgan@charter.net