6359831 2001-04-12 17:24 -0400 /44 rader/ Sylvain Robitaille <syl@ALCOR.CONCORDIA.CA> Sänt av: joel@lysator.liu.se Importerad: 2001-04-13 07:55 av Brevbäraren Extern mottagare: BUGTRAQ@SECURITYFOCUS.COM Externa svar till: syl@ALCOR.CONCORDIA.CA Mottagare: Bugtraq (import) <16507> Kommentar till text 6358568 av William D. Colburn (aka Schlake) <wcolburn@NMT.EDU> Ärende: Re: ntp-4.0.99k23 had been updated ------------------------------------------------------------ From: Sylvain Robitaille <syl@ALCOR.CONCORDIA.CA> To: BUGTRAQ@SECURITYFOCUS.COM Message-ID: <200104122124.RAA31899@alcor.concordia.ca> William Colburn wrote: > The package is newer as of today. True enough, but I have the impression that there are more changes there than just the area of code that's affected by the buffer overflow that triggered the start of this thread. Can anyone confirm this? I've looked and I *believe* the only difference relevant to this discussion between today's ntp-4.0.99k23 and last Friday's is the appended patch. If anyone believes there are more relevant differences could you please point them out? -- ---------------------------------------------------------------------- Sylvain Robitaille syl@alcor.concordia.ca Systems analyst Concordia University Instructional & Information Technology Montreal, Quebec, Canada ---------------------------------------------------------------------- --- ntpd/ntp_control.c.20010412 Mon Apr 9 15:47:20 2001 +++ ntpd/ntp_control.c Thu Apr 12 17:11:47 2001 @@ -1759,9 +1759,11 @@ } if (cp < reqend) cp++; - *tp = '\0'; - while (isspace(*(tp-1))) - *(--tp) = '\0'; + while (tp > buf) { + *tp-- = '\0'; + if (!isspace((int)(*tp))) + break; + } reqpt = cp; *data = buf; return v; (6359831) /Sylvain Robitaille <syl@ALCOR.CONCORDIA.CA>/(Ombruten)