From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: text/plain; charset=windows-1252
Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\))
From: yoann padioleau <aryx.padator@gmail.com>
In-Reply-To: <10C19402-24DC-4B30-AD02-F83E12362D5D@gmail.com>
Date: Tue,  9 Dec 2014 16:24:03 -0800
Content-Transfer-Encoding: quoted-printable
Message-Id: <D0163309-3424-49C5-ABF6-33A58FFD2241@gmail.com>
References: <10C19402-24DC-4B30-AD02-F83E12362D5D@gmail.com>
To: Fans of the OS Plan 9 from Bell Labs <9fans@9fans.net>
Subject: Re: [9fans] memory bug in 5l
Topicbox-Message-UUID: 35f30892-ead9-11e9-9d60-3106f5b1d025

Also, by curiosity, does anybody know why 5a/, 5l/, 5c/
(and the other architecture variants) are redefining
malloc and free? Why not using the malloc and free
from the libc?


On Dec 9, 2014, at 4:21 PM, yoann padioleau <aryx.padator@gmail.com> =
wrote:

> in 5l/obj.c#zaddr()
> there is:
> 	case D_FCONST:
> 		while(nhunk < sizeof(Ieee))
> 			gethunk();
> 		a->ieee =3D (Ieee*)hunk;
> 		nhunk -=3D NSNAME;
> 		hunk +=3D NSNAME;
>=20
> I think it=92s a copy paste bug, it should
> be sizeof(Ieee) instead of those NSNAME
> (or even better the whole code could be factorized
> in a call to a->ieee =3D malloc(sizeof(Ieee));
>=20