Date: Sat, 26 Jul 1997 20:41:54 -0700 (PDT) From: Bill Paul <wpaul@FreeBSD.ORG> To: cvs-committers@FreeBSD.ORG, cvs-all@FreeBSD.ORG, cvs-usrsbin@FreeBSD.ORG Subject: cvs commit: src/usr.sbin/ypserv yp_dnslookup.c Message-ID: <199707270341.UAA26484@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
wpaul 1997/07/26 20:41:54 PDT
Modified files:
usr.sbin/ypserv yp_dnslookup.c
Log:
Fix a bug in the async DNS resolver that can crash ypserv. yp_prune_dnsq()
is not sane: if the TTL on a pending but unanswered query hits 0 and the
circular queue entry is removed and free()d, the for() loop may still try
to use the entry pointer (which now points at no longer valid memory).
usually, deleting only the last entry off the end of the queue worked, but
if more than one was deleted, the server would crash. I changed things a
bit so this shouldn't happen anymore.
Also arranged to call the prune routine a bit more often.
Revision Changes Path
1.11 +9 -4 src/usr.sbin/ypserv/yp_dnslookup.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199707270341.UAA26484>