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: <20201130220505 DOT 0AE4282C54FD AT turkos DOT aspodata DOT se> <7c75ed03-456c-b408-8b50-0448f6b3a04f AT epilitimus DOT com> <1b2c64b3-6a36-c1f3-dd54-bb583c6bea17 AT epilitimus DOT com> <475f980e-fddd-60d1-9a02-a5bc5fb5805b AT epilitimus DOT com> <8706c896-e01a-9e6e-2bab-fe3607064093 AT epilitimus DOT com> From: "Glenn (glimrick AT epilitimus DOT com) [via 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: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Precedence: bulk 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