[dpdk-dev] [PATCH] service: don't walk out of bounds when checking services
Eads, Gage
gage.eads at intel.com
Mon Dec 2 17:16:34 CET 2019
> -----Original Message-----
> From: Aaron Conole <aconole at redhat.com>
> Sent: Tuesday, November 26, 2019 8:56 AM
> To: dev at dpdk.org
> Cc: Van Haaren, Harry <harry.van.haaren at intel.com>; Richardson, Bruce
> <bruce.richardson at intel.com>; Pavan Nikhilesh
> <pbhagavatula at marvell.com>; Eads, Gage <gage.eads at intel.com>; Thomas
> Monjalon <thomas at monjalon.net>; David Marchand
> <dmarchan at redhat.com>
> Subject: [PATCH] service: don't walk out of bounds when checking services
>
> The service_valid call is used without properly bounds checking the input
> parameter. Almost all instances of the service_valid call are inside a for()
> loop that prevents excessive walks, but some of the public APIs don't bounds
> check and will pass invalid arguments.
>
> Prevent this by using SERVICE_GET_OR_ERR_RET where it makes sense, and
> adding a bounds check to one service_valid() use.
>
> Fixes: 8d39d3e237c2 ("service: fix race in service on app lcore function")
> Fixes: e9139a32f6e8 ("service: add function to run on app lcore")
> Fixes: e30dd31847d2 ("service: add mechanism for quiescing")
> Signed-off-by: Aaron Conole <aconole at redhat.com>
Acked-by: Gage Eads <gage.eads at intel.com>
Thanks,
Gage
More information about the dev
mailing list