Date: Thu, 20 Jul 2000 17:56:16 +0100 From: Brian Somers <brian@Awfulhak.org> To: Sheldon Hearn <sheldonh@uunet.co.za> Cc: Brian Somers <brian@Awfulhak.org>, Jordan Hubbard <jkh@FreeBSD.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, Brian Somers <brian@FreeBSD.org>, shin@FreeBSD.org, brian@Awfulhak.org Subject: Re: cvs commit: src/lib/libc/net rcmd.3 rcmd.c Message-ID: <200007201656.RAA03765@hak.lan.Awfulhak.org> In-Reply-To: Message from Sheldon Hearn <sheldonh@uunet.co.za> of "Thu, 20 Jul 2000 18:44:33 %2B0200." <5777.964111473@axl.ops.uunet.co.za>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Thu, 20 Jul 2000 17:31:16 +0100, Brian Somers wrote:
>
> > The don't set *ahost to NULL, it remains pointing to the free()d
> > memory :-(
>
> Can you set it to NULL immediately following the free()? I can't think
> of any reasons why this would be bad.
No, not in the old code. The new version doesn't do a free().
The bug was that rcmd() accidently free()d what it was returning,
before returning.
> Ciao,
> Sheldon.
--
Brian <brian@Awfulhak.org> <brian@[uk.]FreeBSD.org>
<http://www.Awfulhak.org>; <brian@[uk.]OpenBSD.org>
Don't _EVER_ lose your sense of humour !
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200007201656.RAA03765>