From owner-cvs-all Mon Nov 27 12:18:21 2000 Delivered-To: cvs-all@freebsd.org Received: from gndrsh.dnsmgr.net (GndRsh.dnsmgr.net [198.145.92.4]) by hub.freebsd.org (Postfix) with ESMTP id 9C76E37B479; Mon, 27 Nov 2000 12:18:16 -0800 (PST) Received: (from freebsd@localhost) by gndrsh.dnsmgr.net (8.9.3/8.9.3) id MAA37340; Mon, 27 Nov 2000 12:18:10 -0800 (PST) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <200011272018.MAA37340@gndrsh.dnsmgr.net> Subject: Re: cvs commit: src/usr.sbin/cron/cron cron.h In-Reply-To: <200011262221.OAA78674@freefall.freebsd.org> from Kris Kennaway at "Nov 26, 2000 02:21:40 pm" To: kris@FreeBSD.org (Kris Kennaway) Date: Mon, 27 Nov 2000 12:18:10 -0800 (PST) Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org X-Mailer: ELM [version 2.4ME+ PL54 (25)] MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG > kris 2000/11/26 14:21:40 PST > > Modified files: > usr.sbin/cron/cron cron.h > Log: > Correct definition of MAXHOSTNAMELEN in ifdef'ed out code I actaully was ignoring these until it hit me, your actually probably breaking the purpose of these. Old systems that didn't have MAXHOSTNAMELEN defined in system headers had a 64 byte length for this. I suspect if one takes this code after your ``Correction'' and compiles it on one of these systems a buffer overflow condition could easily be triggered. I'd rather just see the ifdef'ed code removed so that the compile fails, rather than the compile working and the code failing in strange ways at run time. -- Rod Grimes - KD7CAX @ CN85sl - (RWG25) rgrimes@gndrsh.dnsmgr.net To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message