From owner-cvs-all Mon Feb 26 0:45:59 2001 Delivered-To: cvs-all@freebsd.org Received: from mailout03.sul.t-online.com (mailout03.sul.t-online.com [194.25.134.81]) by hub.freebsd.org (Postfix) with ESMTP id CC67837B4EC; Mon, 26 Feb 2001 00:45:53 -0800 (PST) (envelope-from garyj@peedub.muc.de) Received: from fwd07.sul.t-online.com by mailout03.sul.t-online.com with smtp id 14XJHz-00024j-09; Mon, 26 Feb 2001 09:45:51 +0100 Received: from peedub.muc.de (320038014727-0001@[62.155.144.9]) by fmrl07.sul.t-online.com with esmtp id 14XJHr-1Kp1IOC; Mon, 26 Feb 2001 09:45:43 +0100 Received: from peedub.muc.de (localhost [127.0.0.1]) by peedub.muc.de (8.11.2/8.11.1) with ESMTP id f1Q8jZu12976; Mon, 26 Feb 2001 09:45:40 +0100 (CET) (envelope-from garyj@peedub.muc.de) Message-Id: <200102260845.f1Q8jZu12976@peedub.muc.de> To: Warner Losh Cc: Gary Jennejohn , cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/usr.sbin/pccard/pccardd util.c Reply-To: Gary Jennejohn In-reply-to: Your message of "Sun, 25 Feb 2001 16:39:01 MST." <200102252339.f1PNd1Y01249@billy-club.village.org> Date: Mon, 26 Feb 2001 09:45:35 +0100 From: Gary Jennejohn X-Sender: 320038014727-0001@t-dialin.net Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG Warner Losh writes: > In message <200102251952.f1PJqhe95166@freefall.freebsd.org> Gary Jennejohn wr > ites: > : Use vsnprintf in logmsg() to avoid overflowing the array on the stack. > : The problem was noted with an older model 3Com 3C589 which seems > : to return more than 256 bytes of data. > > The fix is a good one, but the explaination doesn't make sense to be > because CIS tuples are limited to 254 bytes... > > Warner > > well, all I can say is that sticking the card into the slot caused pccardd to core dump. After applying the fix I noticed that the log output contained lots of junk after the ID strings. Apparently this old card didn't stop at 256 bytes or the kernel was reading too much out of the card. --- Gary Jennejohn / garyj@muc.de gj@freebsd.org To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message