On Fri, Nov 13, 2015 at 02:27:05PM +0100, Sumit Bose wrote:
> +static int sbus_request_valist_check(va_list va, int
first_arg_type)
> +{
> + int ret = EOK;
> +#ifdef HAVE_DBUSBASICVALUE
> + int type;
> + va_list va_check;
> + const DBusBasicValue *value;
> + bool ok;
> +
> + va_copy(va_check, va);
> +
> + type = first_arg_type;
> + while (type != DBUS_TYPE_INVALID) {
> + value = va_arg(va_check, const DBusBasicValue*);
> +
> + if (type == DBUS_TYPE_STRING) {
> + ok = sss_utf8_check((const uint8_t *) value->str,
> + strlen(value->str));
> + if (!ok) {
> + DEBUG(SSSDBG_MINOR_FAILURE,
> + "Back end message [%s] contains invalid non-UTF8
" \
> + "characters");
value->str is missing, additionally sbus_request_return_and_finish() is
used by the responders as well so 'Back end message' should be changed
to 'S-Bus message' or similar.
bye,
Sumit
Oops, sorry..