qemu-ga: Avoiding freeing line prematurely
authorDehan Meng <demeng@redhat.com>
Thu, 7 Nov 2024 10:21:55 +0000 (12:21 +0200)
committerKonstantin Kostiuk <kkostiuk@redhat.com>
Thu, 7 Nov 2024 10:28:20 +0000 (12:28 +0200)
It's now only freed at the end of the function.

Signed-off-by: Dehan Meng <demeng@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Konstantin Kostiuk <kkostiuk@redhat.com>
Message-ID: <20241107102155.57573-4-kkostiuk@redhat.com>
Signed-off-by: Konstantin Kostiuk <kkostiuk@redhat.com>
qga/commands-linux.c

index b0704ad423dee651f899f1f0a72e3950da4d1006..cf077eb03d76eb2209b26f8122700aa1d98dc5c7 100644 (file)
@@ -2138,7 +2138,6 @@ GuestNetworkRouteList *qmp_guest_network_get_route(Error **errp)
         fp = fopen(route_files[i], "r");
         if (fp == NULL) {
             error_setg_errno(errp, errno, "open(\"%s\")", route_files[i]);
-            free(line);
             continue;
         }
 
@@ -2212,9 +2211,9 @@ GuestNetworkRouteList *qmp_guest_network_get_route(Error **errp)
             route = NULL;
         }
 
-        free(line);
         fclose(fp);
     }
 
+    free(line);
     return head;
 }