(Responding to both patches here as I'm having email reception problems) On 3 March 2016 at 15:12, Jarno Rajahalme <ja...@ovn.org> wrote: > Commit 38876d31 fixed setting 'msg' when resizing an ofpbuf, but > failed to fix the same issue in ofpbuf_clone_with_headroom(). Without > this fix the newly cloned ofpbuf's 'msg', if non-NULL, will point to > the buffer of the original ofpbuf. > > Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
In the first patch, there are two changes: Adjusting the header offset calculation, and some nice cosmetic improvements. It's not entirely straightforward to see, but I think that if the ptrdiff_t is unsigned and the new buffer is actually allocated earlier in memory than the old buffer, then the offset is really wrong. This change should stand alone from cosmetic fixes. That said, I completely agree with those changes. This second patch LGTM. Acked-by: Joe Stringer <j...@ovn.org> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
