ofproto-dpif: Restore optimization for no-actions case (without leak).
authorBen Pfaff <blp@nicira.com>
Tue, 10 Jan 2012 23:34:55 +0000 (15:34 -0800)
committerBen Pfaff <blp@nicira.com>
Mon, 16 Jan 2012 20:55:05 +0000 (12:55 -0800)
commit8338659aa5a02421959bece64f6233216de6a101
treeaa5c868d278354dfaaab86e202c83b70cb1fa88f
parent19a79607446350abd1692da317644c36b6df192f
ofproto-dpif: Restore optimization for no-actions case (without leak).

Commit 968131c1809 (ofproto-dpif: Omit "execute" operation entirely when
there are no actions.) introduced an optimization for the case where a
flow translated to ODP actions had no actions at all (i.e. the packet is
to be dropped).  It also introduced a memory leak (the packet was not
freed).

Commit 999fba59afd (ofproto-dpif: Implement PACKET_IN in userspace.)
inadvertently removed the optimization and as a side effect fixed the
memory leak.

This commit restores the optimization but not the memory leak.

Signed-off-by: Ben Pfaff <blp@nicira.com>
ofproto/ofproto-dpif.c