Re: [ovs-dev] [PATCH 2/2] dpif-linux: Fix a bug.

2013-12-17 Thread Jarno Rajahalme
Alex, Thanks for spotting this. I had momentarily missed the fact that ofpbuf_use_stub() sets the size to zero. Pushed to master, Jarno On Dec 17, 2013, at 12:16 AM, Alex Wang wrote: > Commit da546e0 (dpif: Allow execute to modify the packet.) introduced > a bug by subtracting the zero-val

[ovs-dev] [PATCH 2/2] dpif-linux: Fix a bug.

2013-12-17 Thread Alex Wang
Commit da546e0 (dpif: Allow execute to modify the packet.) introduced a bug by subtracting the zero-value ofpbuf size by "sizeof(struct nlattr)" and assigning the result back to the ofpbuf size. This bug causes the ovs-assert failure in facet_push_stats(). This commit fixes the bug by assigning t