Date: Thu, 28 Jan 1999 03:53:36 +0800 From: "Joerg B. Micheel" <joerg@krdl.org.sg> To: Matthew Dillon <dillon@apollo.backplane.com>, current@FreeBSD.ORG Subject: Re: bug in netatm/atm_cm.c ? Message-ID: <19990128035336.V496@krdl.org.sg> In-Reply-To: <199901272215.OAA63874@apollo.backplane.com>; from Matthew Dillon on Wed, Jan 27, 1999 at 02:15:11PM -0800 References: <199901272215.OAA63874@apollo.backplane.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Jan 27, 1999 at 02:15:11PM -0800, Matthew Dillon wrote:
> around line 2192. Shouldn't this be ap->aal.type == ATM_AAL5 ???
It definitely should, as the } else { branch on line 2211 is never reached.
The reason it has not been discovered is that noone hardly ever uses anything
other than AAL5 (as of today) and this code always defaults to it. Just wondering
why nobody ever got a compiler warning with the AAL4 code never reached.
Joerg
--
Joerg B. Micheel Email: <joerg@krdl.org.sg>
SingAREN Technology Center Phone: +65 8742582
Kent Ridge Digital Labs, Rm 3-65, C041 Fax: +65 7744990
21 Heng Mui Keng Terrace Pager: +65 96016020
Singapore 119613 Plan: Troubleshooting ATM
Republic of Singapore Networks and Applications
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?19990128035336.V496>