diff options
| author | Cédric Le Goater <clg@redhat.com> | 2024-04-25 11:02:13 +0200 |
|---|---|---|
| committer | Cédric Le Goater <clg@redhat.com> | 2024-05-16 16:59:19 +0200 |
| commit | 04f8e4f29b5264f855be015852497b3c1dcdb3c4 (patch) | |
| tree | e11e3a2584efc4fb3d756871580317a4eb20c5ce /hw/vfio | |
| parent | cbd470f0aac51103e010e45e1b10cb3bfedb4f99 (diff) | |
| download | qemu-04f8e4f29b5264f855be015852497b3c1dcdb3c4.zip qemu-04f8e4f29b5264f855be015852497b3c1dcdb3c4.tar.gz qemu-04f8e4f29b5264f855be015852497b3c1dcdb3c4.tar.bz2 | |
vfio/ccw: Use g_autofree variable in vfio_ccw_register_irq_notifier()
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Farman <farman@linux.ibm.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>
Diffstat (limited to 'hw/vfio')
| -rw-r--r-- | hw/vfio/ccw.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/hw/vfio/ccw.c b/hw/vfio/ccw.c index 90e4a53..6764388 100644 --- a/hw/vfio/ccw.c +++ b/hw/vfio/ccw.c @@ -384,7 +384,7 @@ static void vfio_ccw_register_irq_notifier(VFIOCCWDevice *vcdev, Error **errp) { VFIODevice *vdev = &vcdev->vdev; - struct vfio_irq_info *irq_info; + g_autofree struct vfio_irq_info *irq_info = NULL; size_t argsz; int fd; EventNotifier *notifier; @@ -421,14 +421,14 @@ static void vfio_ccw_register_irq_notifier(VFIOCCWDevice *vcdev, if (ioctl(vdev->fd, VFIO_DEVICE_GET_IRQ_INFO, irq_info) < 0 || irq_info->count < 1) { error_setg_errno(errp, errno, "vfio: Error getting irq info"); - goto out_free_info; + return; } if (event_notifier_init(notifier, 0)) { error_setg_errno(errp, errno, "vfio: Unable to init event notifier for irq (%d)", irq); - goto out_free_info; + return; } fd = event_notifier_get_fd(notifier); @@ -439,9 +439,6 @@ static void vfio_ccw_register_irq_notifier(VFIOCCWDevice *vcdev, qemu_set_fd_handler(fd, NULL, NULL, vcdev); event_notifier_cleanup(notifier); } - -out_free_info: - g_free(irq_info); } static void vfio_ccw_unregister_irq_notifier(VFIOCCWDevice *vcdev, |