<br><tt><font size=2>Eric Blake <eblake@redhat.com> wrote on 04/06/2010
10:30:16 AM:<br>
<br>
</font></tt>
<br><tt><font size=2>> <br>
> On 04/05/2010 07:27 PM, Stefan Berger wrote:<br>
> > The following rule in direction 'inout'<br>
> > <br>
> > <rule direction='inout' action='drop'><br>
> >   <mac srcmacaddr='1:2:3:4:5:6'/><br>
> > </rule><br>
> > <br>
> > now drops all traffic from and to the given MAC address.<br>
> > So far it would have dropped traffic from the given MAC address<br>
> > and outgoing traffic with the given MAC address, which is not
useful<br>
> > since the packets will always have the VM's MAC address as source<br>
> > MAC address.<br>
> <br>
> Agreed that a bi-directional filter is morally equivalent to filtering<br>
> src on input and dst on output.<br>
> <br>
> > @@ -1783,7 +1802,8 @@ ebtablesCreateRuleInstance(char chainPre<br>
> >                  goto
err_exit;<br>
> >  <br>
> >              virBufferVSprintf(&buf,<br>
> > -                  
       " --ip6-source-port %s %s",<br>
> > +                  
       " %s %s %s",<br>
> > +                  
       (!reverse) ? "--ip6-source-port" :
"--<br>
> ip6-destination-port",<br>
> <br>
> Avoid negative logic; this would be better as:<br>
> <br>
> reverse ? "--ip6-destination-port" : "--ip6-source-port"<br>
> </font></tt>
<br><tt><font size=2>Yes, fixed this everywhere in the meantime...</font></tt>
<br>
<br><tt><font size=2><br>
> > @@ -1912,7 +1934,8 @@ ebiptablesCreateRuleInstance(virConnectP<br>
> >                  
                     
     rule,<br>
> >                  
                     
     ifname,<br>
> >                  
                     
     vars,<br>
> > -                  
                     
   res);<br>
> > +                  
                     
   res,<br>
> > +                  
                     
   0);<br>
> <br>
> s/0/false/, to match the prototype being bool.<br>
> <br>
> ACK, with those tweaks.</font></tt>
<br>
<br><tt><font size=2>Will do and push.</font></tt>
<br>
<br><tt><font size=2>  Thanks. </font></tt>
<br>
<br><tt><font size=2>  Stefan</font></tt>
<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>