On Sun, 31 May 2026 at 16:34, hemanshu.khilari.foss <[email protected]> wrote: > > qdev_get_clocklist() function returns a pointer to the NamedClockList > struct. This function is called in qdev_alias_clock() and the returned > pointer is immediately dereferenced without a null check. > > Passing clock name that doesn't exist to qdev_get_clocklist() is a > programming error, and so this change is not fixing a bug, only making > the reporting of that programming error a bit more helpful and bringing > it in to line with qdev_get_clock_in() and qdev_get_clock_out(). > > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Resolves: https://gitlab.com/qemu-project/qemu/-/work_items/2342 > Signed-off-by: hemanshu.khilari.foss <[email protected]> > ---
Thanks for this patch -- I've applied it to my target-arm.next tree, so it will get upstream in a week or two. -- PMM
