Thread (6 messages) flat view 6 messages, 2 authors, 2012-07-09

Re: [PATCH 1/2] netfilter: ipset: timeout fixing bug broke SET target special timeout value

From: Pablo Neira Ayuso <pablo@netfilter.org>
Date: 2012-07-09 08:58:27
Also in: netfilter-devel

On Mon, Jul 09, 2012 at 12:29:03AM -0700, David Miller wrote:
From: pablo@netfilter.org
Date: Fri,  6 Jul 2012 13:39:38 +0200
quoted
+	if (add_opt.timeout != IPSET_NO_TIMEOUT
+	    && add_opt.timeout > UINT_MAX/MSEC_PER_SEC)
We do not write conditionals like this, with operators beginning
a continued line.  Instead, write this as:

	if (a &&
	    b)
Oops, indeed, sorry. New patch attached.

I've also rebased my tree to include this change. Should I send a new
pull request?

Let me know what you prefer.

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help