These functions can't fail really. Drop checking of their retval then. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> --- src/access/viraccessdriverpolkit.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/access/viraccessdriverpolkit.c b/src/access/viraccessdriverpolkit.c index 496210fabe..5d6c4204ff 100644 --- a/src/access/viraccessdriverpolkit.c +++ b/src/access/viraccessdriverpolkit.c @@ -59,10 +59,8 @@ virAccessDriverPolkitFormatAction(const char *typename, char *actionid = NULL; size_t i; - if (virAsprintf(&actionid, "%s.%s.%s", - VIR_ACCESS_DRIVER_POLKIT_ACTION_PREFIX, - typename, permname) < 0) - return NULL; + virAsprintf(&actionid, "%s.%s.%s", VIR_ACCESS_DRIVER_POLKIT_ACTION_PREFIX, + typename, permname); for (i = 0; actionid[i]; i++) if (actionid[i] == '_') -- 2.21.0 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list