Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 15 Jan 2002 09:28:24 -0800
From:      Luigi Rizzo <rizzo@icir.org>
To:        Ruslan Ermilov <ru@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sbin/natd natd.c
Message-ID:  <20020115092824.B75791@iguana.icir.org>
In-Reply-To: <200201151707.g0FH7uB73802@freefall.freebsd.org>
References:  <200201151707.g0FH7uB73802@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jan 15, 2002 at 09:07:56AM -0800, Ruslan Ermilov wrote:
> ru          2002/01/15 09:07:56 PST
> 
>   Modified files:
>     sbin/natd            natd.c 
>   Log:
>   Back out part of the revision 1.2 changes -- sendto(2) can
>   not return ENOBUFS for unreliable protocols like divert.

actually sendto does return ENOBUFS, it's just select() which
cannot be used to check for writability on such sockets.

>   This should fix an issue when natd(8) keeps spamming already
>   full dummynet(4) queues with the same packet forever.

not just dummynet, this applies to _any_ device-level queue,
including ethernet, ppp and serial lines.

>   Spotted by:     chkno@dork.com
>   Explained by:   luigi

probably not well enough given the commit message :)
(just kidding... thanks to Ruslan for fixing this!)

	cheers
	luigi

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020115092824.B75791>