* [PATCH v3 0/5] ipset: add "inner" flag support
@ 2013-06-29 21:32 Dash Four
2013-07-05 18:06 ` Jozsef Kadlecsik
0 siblings, 1 reply; 4+ messages in thread
From: Dash Four @ 2013-06-29 21:32 UTC (permalink / raw)
To: Jozsef Kadlecsik; +Cc: Pablo Neira Ayuso, Netfilter Core Team
This series of 5 patches implements "inner" flag option in the set
iptables match, allowing matching based on the properties
(source/destination IP address, protocol, port and so on) of the
original (inner) connection in the event of the following
ICMP[v4,v6] messages:
ICMPv4 destination-unreachable (code 3);
ICMPv4 source-quench (code 4);
ICMPv4 time-exceeded (code 11);
ICMPv6 destination-unreachable (code 1);
ICMPv6 packet-too-big (code 2);
ICMPv6 time-exceeded (code 3);
Revision history:
v1 * initial revision
v2 * redundant code removed;
* added a new header file (ip_set_icmp.h) with 2 inline functions,
allowing access to the internal icmp header properties;
* removed ip[46]inneraddr[ptr]functions as they are no longer needed
* added new ipv[46]addr[ptr] and ip_set_get*port functions, the old
functions are still preserved for backwards compatibility
v3 * rename and move ip_set_get_icmpv[46]_inner_hdr functions to
ip_set_core.c and remove ip_set_icmp.h
* move icmpv[46] protocol and offset checks inside
ip_set_get_ip[46]_inner_hdr functions
* eliminate ip[46]addrptr & ip_set_get_ip[46]_port backward-compatible
functions and rename the new ones to use the same name
* eliminate single-path error gotos in ip_set.h and ip_set_getport.c
Dash Four (5):
iptables: bugfix: prevent wrong syntax being accepted by the set match
ipset: add "inner" flag implementation
ipset: add set match "inner" flag support
iptables: add set match "inner" flag support
iptables (userspace): add set match "inner" flag support
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3 0/5] ipset: add "inner" flag support
2013-06-29 21:32 [PATCH v3 0/5] ipset: add "inner" flag support Dash Four
@ 2013-07-05 18:06 ` Jozsef Kadlecsik
2013-07-05 20:33 ` Dash Four
0 siblings, 1 reply; 4+ messages in thread
From: Jozsef Kadlecsik @ 2013-07-05 18:06 UTC (permalink / raw)
To: Dash Four; +Cc: Pablo Neira Ayuso, Netfilter Core Team
Hello,
On Sat, 29 Jun 2013, Dash Four wrote:
> This series of 5 patches implements "inner" flag option in the set
> iptables match, allowing matching based on the properties
> (source/destination IP address, protocol, port and so on) of the
> original (inner) connection in the event of the following
> ICMP[v4,v6] messages:
>
> ICMPv4 destination-unreachable (code 3);
> ICMPv4 source-quench (code 4);
> ICMPv4 time-exceeded (code 11);
> ICMPv6 destination-unreachable (code 1);
> ICMPv6 packet-too-big (code 2);
> ICMPv6 time-exceeded (code 3);
>
> Revision history:
>
> v1 * initial revision
> v2 * redundant code removed;
> * added a new header file (ip_set_icmp.h) with 2 inline functions,
> allowing access to the internal icmp header properties;
> * removed ip[46]inneraddr[ptr]functions as they are no longer needed
> * added new ipv[46]addr[ptr] and ip_set_get*port functions, the old
> functions are still preserved for backwards compatibility
> v3 * rename and move ip_set_get_icmpv[46]_inner_hdr functions to
> ip_set_core.c and remove ip_set_icmp.h
> * move icmpv[46] protocol and offset checks inside
> ip_set_get_ip[46]_inner_hdr functions
> * eliminate ip[46]addrptr & ip_set_get_ip[46]_port backward-compatible
> functions and rename the new ones to use the same name
> * eliminate single-path error gotos in ip_set.h and ip_set_getport.c
>
> Dash Four (5):
> iptables: bugfix: prevent wrong syntax being accepted by the set match
> ipset: add "inner" flag implementation
> ipset: add set match "inner" flag support
> iptables: add set match "inner" flag support
> iptables (userspace): add set match "inner" flag support
I have just noticed one single issue with your patches: the revision
bumping for the set types are missing.
So please extend the second patch with the type revisions incremented
(have a look at the hash:net type how the revision comments are kept)
and add a userspace patch with the revisions bumped in userspace too (it's
fine to copy the most recent ones and increase the revisions, adjust the
descriptions). That's all and then the patchset is ready for inclusion.
Best regards,
Jozsef
-
E-mail : kadlec@blackhole.kfki.hu, kadlecsik.jozsef@wigner.mta.hu
PGP key : http://www.kfki.hu/~kadlec/pgp_public_key.txt
Address : Wigner Research Centre for Physics, Hungarian Academy of Sciences
H-1525 Budapest 114, POB. 49, Hungary
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3 0/5] ipset: add "inner" flag support
2013-07-05 18:06 ` Jozsef Kadlecsik
@ 2013-07-05 20:33 ` Dash Four
2013-07-05 20:48 ` Jozsef Kadlecsik
0 siblings, 1 reply; 4+ messages in thread
From: Dash Four @ 2013-07-05 20:33 UTC (permalink / raw)
To: Jozsef Kadlecsik; +Cc: Pablo Neira Ayuso, Netfilter Core Team
Jozsef Kadlecsik wrote:
> I have just noticed one single issue with your patches: the revision
> bumping for the set types are missing.
>
Interesting. I wasn't aware you've introduced this feature and by
looking at the logs, this has been present since 6.14 - shame on me for
not noticing it.
> So please extend the second patch with the type revisions incremented
> (have a look at the hash:net type how the revision comments are kept)
> and add a userspace patch with the revisions bumped in userspace too (it's
> fine to copy the most recent ones and increase the revisions, adjust the
> descriptions).
In other words, something like:
kernel/net/netfilter/ip_set_*.c
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-#define IPSET_TYPE_REV_MAX 3 /* Counters support added */
+/* 3 Counters support added */
+#define IPSET_TYPE_REV_MAX 4 /* Inner flag support added */
lib/ipset_*.c
~~~~~~~~~~~~~
copy the entire ipset_hash_netX struct, bump the ".revision" and adjust
the ".description" fields?
Would that suffice?
Also, would you like me to modify my 3rd patch (which is where I've made
the changes to all sets - in patch two I reimplemented the "internal"
functions) and issue a separate one for all lib/ipset_*.c changes or
would you like me to reissue the whole set of patches (1-6)?
> That's all and then the patchset is ready for inclusion.
>
Thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v3 0/5] ipset: add "inner" flag support
2013-07-05 20:33 ` Dash Four
@ 2013-07-05 20:48 ` Jozsef Kadlecsik
0 siblings, 0 replies; 4+ messages in thread
From: Jozsef Kadlecsik @ 2013-07-05 20:48 UTC (permalink / raw)
To: Dash Four; +Cc: Pablo Neira Ayuso, Netfilter Core Team
On Fri, 5 Jul 2013, Dash Four wrote:
> Jozsef Kadlecsik wrote:
> > I have just noticed one single issue with your patches: the revision bumping
> > for the set types are missing.
> >
> Interesting. I wasn't aware you've introduced this feature and by
> looking at the logs, this has been present since 6.14 - shame on me for
> not noticing it.
>
> > So please extend the second patch with the type revisions incremented (have
> > a look at the hash:net type how the revision comments are kept)
> > and add a userspace patch with the revisions bumped in userspace too (it's
> > fine to copy the most recent ones and increase the revisions, adjust the
> > descriptions).
> In other words, something like:
>
> kernel/net/netfilter/ip_set_*.c
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> -#define IPSET_TYPE_REV_MAX 3 /* Counters support added */
> +/* 3 Counters support added */
> +#define IPSET_TYPE_REV_MAX 4 /* Inner flag support added */
>
> lib/ipset_*.c
> ~~~~~~~~~~~~~
> copy the entire ipset_hash_netX struct, bump the ".revision" and adjust the
> ".description" fields?
>
> Would that suffice?
Yes, exactly.
> Also, would you like me to modify my 3rd patch (which is where I've made
> the changes to all sets - in patch two I reimplemented the "internal"
> functions) and issue a separate one for all lib/ipset_*.c changes or
> would you like me to reissue the whole set of patches (1-6)?
I meant the third patch, but wrote "second" above....
You can reissue just the third and send the new sixth for lib/ipset_*.c.
Best regards,
Jozsef
-
E-mail : kadlec@blackhole.kfki.hu, kadlecsik.jozsef@wigner.mta.hu
PGP key : http://www.kfki.hu/~kadlec/pgp_public_key.txt
Address : Wigner Research Centre for Physics, Hungarian Academy of Sciences
H-1525 Budapest 114, POB. 49, Hungary
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-07-05 20:48 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-29 21:32 [PATCH v3 0/5] ipset: add "inner" flag support Dash Four
2013-07-05 18:06 ` Jozsef Kadlecsik
2013-07-05 20:33 ` Dash Four
2013-07-05 20:48 ` Jozsef Kadlecsik
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.