On 9/21/26 2:43 PM, Eric Farman wrote: > The loop to store pending IRQs uses g_try_realloc() to expand the > buffer being used, but the returned address is stored in a > pointer-to-pointer, thus the check is supposed to be examining > the contents of the pointer, and not the pointer itself. > > Define a temporary variable to catch and test the returned pointer. > If it's NULL, return to the caller to perform the free on the > unmodified buffer. If it's not NULL, store it in the caller's > pointer and restart the loop against the new struct. > > Cc: [email protected] > Fixes: 3a553fc658 ("s390x/kvm: implement floating-interrupt controller > device") > Signed-off-by: Eric Farman <[email protected]> With Christian's suggestion addressed:
Reviewed-by: Matthew Rosato <[email protected]>
