On 15 November 2016 at 16:37, Ben Pfaff <[email protected]> wrote:
> On Tue, Nov 15, 2016 at 04:15:04PM -0800, Joe Stringer wrote:
>> netdev_get_vports() previously counted the number of ports outside the
>> mutex, allocated enough memory for that number, then grabbed the mutex
>> to iterate through them and filled the array with the pointers.
>>
>> This is logically wrong; in theory the number of ports could change
>> between allocating the memory and grabbing the mutex. In practice, only
>> the main thread manages these so there is no chance for a segfault. Fix
>> it up anyway.
>>
>> Signed-off-by: Joe Stringer <[email protected]>
>
> Acked-by: Ben Pfaff <[email protected]>

Thanks, applied.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to