On 03/02/2010 10:04 AM, Jim Meyering wrote:
Eric Blake wrote:All other uses of get_str_prop in this file that ignored failure explicitly cast to void. * src/node_device/node_device_hal.c (dev_create): Silence coverity warning. --- src/node_device/node_device_hal.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/node_device/node_device_hal.c b/src/node_device/node_device_hal.c index 6cc2864..4bf445d 100644 --- a/src/node_device/node_device_hal.c +++ b/src/node_device/node_device_hal.c @@ -463,7 +463,7 @@ static void dev_create(const char *udi) goto cleanup; /* Some devices don't have a path in sysfs, so ignore failure */ - get_str_prop(ctx, udi, "linux.sysfs_path",&devicePath); + (void)get_str_prop(ctx, udi, "linux.sysfs_path",&devicePath); dev = virNodeDeviceAssignDef(&driverState->devs, def);Confirmed. That's the only one that's ignored, and your change makes it consistent with the others. ACK.
+1 Dave -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list