Looks good thanks. Ethan
On Fri, Jul 20, 2012 at 4:25 PM, Ben Pfaff <b...@nicira.com> wrote: > 'packet' is already a struct ofpbuf *. > --- > ofproto/ofproto-dpif.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c > index dc15c15..6bc2cf4 100644 > --- a/ofproto/ofproto-dpif.c > +++ b/ofproto/ofproto-dpif.c > @@ -4725,7 +4725,7 @@ send_packet(const struct ofport_dpif *ofport, struct > ofpbuf *packet) > struct flow flow; > int error; > > - flow_extract((struct ofpbuf *) packet, 0, 0, 0, &flow); > + flow_extract(packet, 0, 0, 0, &flow); > odp_port = vsp_realdev_to_vlandev(ofproto, ofport->odp_port, > flow.vlan_tci); > if (odp_port != ofport->odp_port) { > -- > 1.7.2.5 > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev