On Wed, May 5, 2021 at 5:36 PM Bruce Richardson <bruce.richard...@intel.com> wrote: > > On Wed, May 05, 2021 at 03:22:48PM +0000, Ciara Power wrote: > > The list_commands() function accessed the callbacks list, > > but did not take the lock. This may have caused inconsistencies if > > callbacks were being registered at the same time. > > This is now fixed to lock before iterating the list, > > and unlock afterwards. > > > > Fixes: f38748736eb2 ("telemetry: add default callback commands") > > Cc: sta...@dpdk.org > > > > Reported-by: David Marchand <david.march...@redhat.com> > > Signed-off-by: Ciara Power <ciara.po...@intel.com> > Acked-by: Bruce Richardson <bruce.richard...@intel.com>
Applied, thanks. -- David Marchand