NuKVM/libvirt
3
0
Fork 0
mirror of https://github.com/libvirt/libvirt.git synced 2025-02-25 18:55:26 -06:00
Code Issues Packages Projects Releases Wiki Activity

virPolkitCheckAuth: Avoid virStringListLength in loop condition

Browse Source 
Don't re-calculate the string list length on every iteration. Convert
the loop to NULL-terminated iteration.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Peter Krempa 2021-02-05 17:53:05 +01:00
parent 2f9b2c0cdd
commit 1114cf5e7e
1 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/util/virpolkit.c
View File

@ -75,7 +75,7 @@ int virPolkitCheckAuth(const char *actionid,
gboolean is_authorized; gboolean is_authorized;
gboolean is_challenge; gboolean is_challenge;
bool is_dismissed = false; bool is_dismissed = false;
size_t i; const char **next;
if (!(sysbus = virGDBusGetSystemBus())) if (!(sysbus = virGDBusGetSystemBus()))
return -1; return -1;
@ -90,8 +90,15 @@ int virPolkitCheckAuth(const char *actionid,
gprocess = g_variant_builder_end(&builder); gprocess = g_variant_builder_end(&builder);
g_variant_builder_init(&builder, G_VARIANT_TYPE("a{ss}")); g_variant_builder_init(&builder, G_VARIANT_TYPE("a{ss}"));
for (i = 0; i < virStringListLength(details); i += 2)
g_variant_builder_add(&builder, "{ss}", details[i], details[i + 1]); if (details) {
for (next = details; *next; next++) {
const char *detail1 = *(next++);
const char *detail2 = *next;
g_variant_builder_add(&builder, "{ss}", detail1, detail2);
}
}
gdetails = g_variant_builder_end(&builder); gdetails = g_variant_builder_end(&builder);
message = g_variant_new("((s@a{sv})s@a{ss}us)", message = g_variant_new("((s@a{sv})s@a{ss}us)",