On Fri, Jun 23, 2023 at 11:22 AM Oleg Vasilev <oleg.vasi...@virtuozzo.com>
wrote:

> If there are no parameters, there is nothing to validate.
> If params == NULL, memcpy below results in memcpy(sorted, NULL, 0),
> which is UB.
>
> Found by UBSAN. Example of this codepath: virDomainBlockCopy()
> (where nparams == 0 is valid) -> qemuDomainBlockCopy()
>
> Signed-off-by: Oleg Vasilev <oleg.vasi...@virtuozzo.com>
> ---
>  src/util/virtypedparam.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
>
Reviewed-by: Kristina Hanicova <khani...@redhat.com>

Kristina

Reply via email to