<br><tt><font size=2>Eric Blake <eblake@redhat.com> wrote on 03/18/2010
03:29:17 PM:<br>
<br>
> On 03/18/2010 09:18 AM, Stefan Berger wrote:<br>
> > This patch adds support for L3/L4 filtering using iptables. This
adds<br>
> > support for 'tcp', 'udp', 'icmp', 'igmp', 'sctp' etc. filtering.<br>
> > <br>
> > As mentioned in the introduction, a .c file provided by this
patch<br>
> > is #include'd into a .c file. This will need work, but should
be alright<br>
> > for review.<br>
> > <br>
> > Signed-off-by: Stefan Berger <stefanb@us.ibm.com><br>
> <br>
> [same caveats as earlier in this thread...]<br>
> <br>
> > +                  
   unsigned int priority,<br>
> > +                  
   int isIptablesRule)<br>
> >  {<br>
> >      ebiptablesRuleInstPtr inst;<br>
> >  <br>
> > @@ -225,6 +229,7 @@ ebiptablesAddRuleInst(virConnectPtr conn<br>
> >      inst->neededProtocolChain = neededChain;<br>
> >      inst->chainprefix = chainprefix;<br>
> >      inst->priority = priority;<br>
> > +    inst->isIptablesRule = isIptablesRule;<br>
> <br>
> This should be bool, not int.</font></tt>
<br>
<br><tt><font size=2>Changed this now.</font></tt>
<br>
<br><tt><font size=2>Thanks and regards,</font></tt>
<br><tt><font size=2>   Stefan</font></tt>
<br>
<br><tt><font size=2><br>
> <br>
> -- <br>
> Eric Blake   eblake@redhat.com    +1-801-349-2682<br>
> Libvirt virtualization library </font></tt><a href=http://libvirt.org/><tt><font size=2>http://libvirt.org</font></tt></a><tt><font size=2><br>
> <br>
> [attachment "signature.asc" deleted by Stefan Berger/Watson/IBM]
</font></tt>