Re: [ovs-dev] [PATCH] ofp-util: Reload ofpbuf pointers after nx_put_match().

2012-07-12 Thread Ben Pfaff
Thank you, I pushed this to master. On Thu, Jul 12, 2012 at 12:46:36PM -0700, Pravin Shelar wrote: > Looks good. > > Thanks, > Pravin. > > On Thu, Jul 12, 2012 at 11:27 AM, Ben Pfaff wrote: > > Fixes a wild pointer write when nx_put_match() expands the ofpbuf. > > > > The change to the OFPST_FL

Re: [ovs-dev] [PATCH] ofp-util: Reload ofpbuf pointers after nx_put_match().

2012-07-12 Thread Pravin Shelar
Looks good. Thanks, Pravin. On Thu, Jul 12, 2012 at 11:27 AM, Ben Pfaff wrote: > Fixes a wild pointer write when nx_put_match() expands the ofpbuf. > > The change to the OFPST_FLOW case isn't strictly a bug fix, it just makes > the code for that case more closely resemble the NXST_FLOW case. > >

[ovs-dev] [PATCH] ofp-util: Reload ofpbuf pointers after nx_put_match().

2012-07-12 Thread Ben Pfaff
Fixes a wild pointer write when nx_put_match() expands the ofpbuf. The change to the OFPST_FLOW case isn't strictly a bug fix, it just makes the code for that case more closely resemble the NXST_FLOW case. Bug #12403. Bug #12460. Bug #12461. Reported-by: Timothy Chen Reported-by: Natasha Gude R