vswitchd: Minor code simplification.
authorBen Pfaff <blp@nicira.com>
Thu, 1 Jan 2009 00:43:28 +0000 (16:43 -0800)
committerBen Pfaff <blp@nicira.com>
Thu, 1 Jan 2009 19:13:54 +0000 (11:13 -0800)
The 'vlan' argument is exactly what we want here.

vswitchd/bridge.c

index 81806d97ecb4d3c5153cd8aa1f8e56fe51f48172..cd92ecd12ffc695c5f718efb6bb5b41c6348a5a8 100644 (file)
@@ -1391,8 +1391,7 @@ compose_dsts(const struct bridge *br, const struct flow *flow, uint16_t vlan,
 
     while (mirrors) {
         struct mirror *m = br->mirrors[mirror_mask_ffs(mirrors) - 1];
-        if ((!m->n_vlans
-             || vlan_is_mirrored(m, in_port->vlan ? in_port->vlan : vlan))
+        if ((!m->n_vlans || vlan_is_mirrored(m, vlan))
             && set_dst(dst, flow, in_port, m->out_port, tags)) {
             if (m->set_vlan >= 0) {
                 dst->vlan = m->set_vlan;