[SSSD] [PATCH] sysdb_set_entry_attr: dump attributes and values on error
Pavel Reichl
preichl at redhat.com
Mon Mar 9 10:24:31 UTC 2015
On 03/09/2015 10:13 AM, Sumit Bose wrote:
> Hi,
>
> this is a patch I have in my tree for some time for debugging purposes.
> Recently I've seen some bug reports with "ldb_modify failed" messages in
> the logs and I think this patch might help in those cases.
>
> Additionally I wonder if SSSDBG_MINOR_FAILURE is a suitable debug level
> here. In most cases an error here prevents a user or group object to be
> written to the cache, so maybe SSSDBG_CRIT_FAILURE is justified here?
>
> bye,
> Sumit
>
>
> _______________________________________________
> sssd-devel mailing list
> sssd-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
Hello Sumit,
when compiling at my environment, I'm seeing this warning:
../src/db/sysdb_ops.c: In function 'sysdb_set_entry_attr':
../src/db/sysdb_ops.c:717:21: warning: format '%u' expects argument of
type 'unsigned int', but argument 6 has type 'size_t' [-Wformat=]
DEBUG(SSSDBG_MINOR_FAILURE, " Value[%u]: [%s].\n",
Would you consider amending the patch?
- DEBUG(SSSDBG_MINOR_FAILURE, " Value[%u]: [%s].\n",
+ DEBUG(SSSDBG_MINOR_FAILURE, " Value[%zu]: [%s].\n",
This seem to remove the warning for me.
Thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20150309/a3cbb3be/attachment.html>
More information about the sssd-devel
mailing list