Re: [libvirt] [PATCH 1/2] Add functions for checking if user or group exists

Instead of duplicating the code from virGet{User,Group}IDByName(), which are static anyway, extend those functions to accept NULL pointers for the result and a boolean for controlling the error reporting. Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com&gt; --- Feel free to suggest any other way of doing this, I just felt like this is the most readable way of doing things src/libvirt_private.syms | 2 ++ src/util/virutil.c | 40 ++++++++++++++++++++++++++++++++-------- src/util/virutil.h | 4 ++++ 3 files changed, 38 insertions(+), 8 deletions(-)

+/* Silently checks if Group @name exists. + * Returns if the group exists and fallbacks to false on error. + */ +int +virDoesGroupExist(const char *name)

+{ + return virGetGroupIDByName(name, NULL, true) == 0; +} + /* Compute the list of primary and supplementary groups associated * with @uid, and including @gid in the list (unless it is -1),