Thanks a lot for the updated patch. I was really only expecting a "yes or no" answer so that I could update the patch description myself, but your quick v2 saved me the trouble so thanks again.
On Wed, Sep 04, 2013 at 09:01:43AM -0700, Guolin Yang, VMware wrote: > You are right, I send anew patch with proper description > > On Wed, Sep 4, 2013 at 8:00 AM, Ben Pfaff <b...@nicira.com> wrote: > > > On Wed, Sep 04, 2013 at 12:36:50AM -0700, gy...@nicira.com wrote: > > > From: Guolin Yang <gy...@nicira.com> > > > > > > In cfm, when allocating the rmp array, the size is not calculated > > correctly > > > which will cause memory corruption. > > > > > > Signed-off-by: Guolin Yang <gy...@nicira.com> > > > > Thanks for the patch. It is obviously correct. > > > > But I am not sure that there is any real memory corruption. Obviously > > there would be, if any of the callers modified the allocated memory. > > But I do not see any of the chain of callers doing that: I only see them > > reading it and eventually freeing it. This will not corrupt any > > memory. Do you see anything different? > > > > Thanks, > > > > Ben. > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
