<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    <br>
    <div class="moz-cite-prefix">On 03/09/2015 12:03 PM, Sumit Bose
      wrote:<br>
    </div>
    <blockquote cite="mid:20150309110319.GG14649@p.redhat.com"
      type="cite">
      <pre wrap="">On Mon, Mar 09, 2015 at 11:24:31AM +0100, Pavel Reichl wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">

On 03/09/2015 10:13 AM, Sumit Bose wrote:
</pre>
        <blockquote type="cite">
          <pre wrap="">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
<a class="moz-txt-link-abbreviated" href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a>
<a class="moz-txt-link-freetext" href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a>
</pre>
        </blockquote>
        <pre wrap="">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",
</pre>
      </blockquote>
      <pre wrap="">
Of course, thank you for catching this, new version attached.

bye,
Sumit

</pre>
      <blockquote type="cite">
        <pre wrap="">
This seem to remove the warning for me.

Thanks!

</pre>
        <br>
        <fieldset class="mimeAttachmentHeader"></fieldset>
        <br>
        <pre wrap="">_______________________________________________
sssd-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a>
<a class="moz-txt-link-freetext" href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a>
</pre>
      </blockquote>
    </blockquote>
    Thanks:<br>
    <br>
    ci: failed on RHEL 7 on test-negcache - I think this is a false
    positive. <br>
    <a class="moz-txt-link-freetext"
      href="http://sssd-ci.duckdns.org/logs/job/9/05/summary.html">http://sssd-ci.duckdns.org/logs/job/9/05/summary.html</a><br>
    <br>
    ACK<br>
  </body>
</html>