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

Reply via email to