On Mon, Nov 03, 2025 at 10:52:07AM +0000, Pranjal Shrivastava wrote:
> > -   if (device->ops->get_region_info_caps) {
> > -           struct vfio_info_cap caps = { .buf = NULL, .size = 0 };
> > +   ret = device->ops->get_region_info_caps(device, &info, &caps);
> > +   if (ret)
> > +           return ret;
> 
> Shall we kfree(caps.buf); before returning?

Yes, I changed it to use a goto unwind

Thanks,
Jason

Reply via email to