X-Authentication-Warning:	delorie.com: mail set sender to geda-user-bounces using -f
X-Recipient:	geda-user AT delorie DOT com
X-CMAE-Analysis:	v=2.4 cv=UZGU9IeN c=1 sm=1 tr=0 ts=5fd4460e
	a=+cj0cO56Fp8x7EdhTra87A==:117 a=uQnqBw2apslg1eJ8GR+KCw==:17
	a=9+rZDBEiDlHhcck0kWbJtElFXBc=:19 a=IkcTkHD0fZMA:10 a=zTNgK-yGK50A:10
	a=KqeppcUizBZ3MAtbRgwA:9 a=QEXdDO2ut3YA:10
X-SECURESERVER-ACCT:	glimrick AT epilitimus DOT com
Subject:	Re: [geda-user] Memory leak patches (was: SAB processing patches)
To:	geda-user AT delorie DOT com
References:	<e4ff3c96-939b-a93e-a32f-5e938b6daa63 AT epilitimus DOT com>
	<alpine DOT DEB DOT 2 DOT 21 DOT 2011302045040 DOT 2894 AT nimbus>
	<20201130220505 DOT 0AE4282C54FD AT turkos DOT aspodata DOT se>
	<7c75ed03-456c-b408-8b50-0448f6b3a04f AT epilitimus DOT com>
	<alpine DOT DEB DOT 2 DOT 21 DOT 2012011606400 DOT 6785 AT nimbus>
	<1b2c64b3-6a36-c1f3-dd54-bb583c6bea17 AT epilitimus DOT com>
	<alpine DOT DEB DOT 2 DOT 21 DOT 2012021253440 DOT 1262 AT nimbus>
	<475f980e-fddd-60d1-9a02-a5bc5fb5805b AT epilitimus DOT com>
	<alpine DOT DEB DOT 2 DOT 21 DOT 2012071333290 DOT 3907 AT nimbus>
	<e6c5135f-b127-fd55-91b2-79aa357a5a07 AT epilitimus DOT com>
	<alpine DOT DEB DOT 2 DOT 21 DOT 2012081644480 DOT 3649 AT nimbus>
	<8706c896-e01a-9e6e-2bab-fe3607064093 AT epilitimus DOT com>
	<alpine DOT DEB DOT 2 DOT 21 DOT 2012111645180 DOT 13096 AT nimbus>
From:	"Glenn (glimrick AT epilitimus DOT com) [via geda-user AT delorie DOT com]" <geda-user AT delorie DOT com>
Message-ID:	<63273f86-b0fd-8835-a13e-6ab408f06dfc@epilitimus.com>
Date:	Fri, 11 Dec 2020 20:24:40 -0800
User-Agent:	Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
	Firefox/60.0 SeaMonkey/2.53.3
MIME-Version:	1.0
In-Reply-To:	<alpine.DEB.2.21.2012111645180.13096@nimbus>
X-AntiAbuse:	This header was added to track abuse, please include it with any abuse report
X-AntiAbuse:	Primary Hostname - a2plcpnl0121.prod.iad2.secureserver.net
X-AntiAbuse:	Original Domain - delorie.com
X-AntiAbuse:	Originator/Caller UID/GID - [47 12] / [47 12]
X-AntiAbuse:	Sender Address Domain - epilitimus.com
X-Get-Message-Sender-Via:	a2plcpnl0121.prod.iad2.secureserver.net: authenticated_id: glimrick AT epilitimus DOT com
X-Authenticated-Sender:	a2plcpnl0121.prod.iad2.secureserver.net: glimrick AT epilitimus DOT com
X-Source:
X-Source-Args:
X-Source-Dir:
X-CMAE-Envelope:	MS4xfEBSOBNmodObwWKh34O/JqcETHVg44TMZGSgXvKeA1QyYIa5T205KR1eHcvscMVgTnNL/4QowJ0BSPQZMfVZDDzpTC23xtkYfu/ueEl8F6NAwNpB1Th/
	DfQ8Hh9akbpKbVgiLe/Nb5eNmvc0G7jUh0kF9gBUs0furK8DX1h//iGchYxagMxw4O606kGqa5NKLee6kw/xzkuPmOhZaNXQG/AlA3YV3adV6Id4CPYJ3ldt
Reply-To:	geda-user AT delorie DOT com
Errors-To:	nobody AT delorie DOT com
X-Mailing-List:	geda-user AT delorie DOT com
X-Unsubscribes-To:	listserv AT delorie DOT com

Roland Lutz wrote:
> A few of the variables you freed were actually still in use, so I had
> to change a few things.  In particular, g_object_set_data takes a
> generic pointer, so the string mustn't be freed while the object is
> still around. 
I looked at this again. While I am sure you are correct about
g_object_data valgrind lists the string as definitely lost so you need
to find another place to free it once you are done with it.
> you accidentally freed the configuration context instead of the
> retrieved string.
yep, don't know what I was thinking. I was obviously freeing the wrong
variable.

Glenn

- Raw text -