Date: Tue, 11 Jul 2000 02:31:19 -0700 (PDT) From: Peter Wemm <peter@FreeBSD.org> To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/lib/libc/net rcmd.c Message-ID: <200007110931.CAA36697@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
peter 2000/07/11 02:31:19 PDT
Modified files:
lib/libc/net rcmd.c
Log:
Try and fix the worst of some highly bogus malloc/free resource
management involving rcmd_af(), getaddrinfo(), freeaddrinfo(), etc.
We set *ahost to point to ai->canonname; and later free the ai-> stuff
and still leave the old pointers in *ahost to the freed data.
Perhaps the best way to deal with this is a static buffer or a static
strdup() that is freed on the next iteration or something. This gives
me headaches just thinking about this.
The new 'AJ' default for malloc() tripped this up.
Revision Changes Path
1.26 +2 -2 src/lib/libc/net/rcmd.c
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?200007110931.CAA36697>