<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 06/11/2013 04:04 AM, Jakub Hrozek wrote:
    <blockquote cite="mid:20130611080406.GP20676@hendrix" type="cite">
      <pre wrap="">On Tue, Jun 11, 2013 at 09:11:58AM +0200, Sumit Bose wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">On Mon, Jun 10, 2013 at 09:30:43PM +0200, Jakub Hrozek wrote:
</pre>
        <blockquote type="cite">
          <pre wrap="">Another Coverity issue.
</pre>
        </blockquote>
        <pre wrap="">
</pre>
        <blockquote type="cite">
          <pre wrap="">From b01b9bc392cd21e72cdb04fa3893091fcac0f8e0 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <a class="moz-txt-link-rfc2396E" href="mailto:jhrozek@redhat.com">&lt;jhrozek@redhat.com&gt;</a>
Date: Mon, 10 Jun 2013 21:29:25 +0200
Subject: [PATCH] Fix allocation check in the AD provider

<a class="moz-txt-link-freetext" href="https://fedorahosted.org/sssd/ticket/1976">https://fedorahosted.org/sssd/ticket/1976</a>
---
 src/providers/ad/ad_init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/providers/ad/ad_init.c b/src/providers/ad/ad_init.c
index 09ba384be6d22ac57f8ac82a4443dc432b41d463..f3c1609dedb6d1ad650b598da53546dce6cf6345 100644
--- a/src/providers/ad/ad_init.c
+++ b/src/providers/ad/ad_init.c
@@ -143,7 +143,7 @@ sssm_ad_id_init(struct be_ctx *bectx,
     ad_ctx-&gt;ldap_ctx = sdap_ctx-&gt;conn;
 
     ad_ctx-&gt;gc_ctx = sdap_id_ctx_conn_add(sdap_ctx, ad_options-&gt;service-&gt;gc);
-    if (sdap_ctx == NULL) {
+    if (sdap_ctx-&gt;gc_ctx == NULL) {
</pre>
        </blockquote>
        <pre wrap="">
I guess you meant ad_ctx-&gt;gc_ctx == NULL ?

bye,
Sumit
</pre>
      </blockquote>
      <pre wrap="">
This is exactly why I prefer reviews even for one-liner patches (any why
I should start running Coverity fixes through Coverity prior to sending
them out).</pre>
    </blockquote>
    <br>
    What prevents us from doing it?<br>
    <br>
    <blockquote cite="mid:20130611080406.GP20676@hendrix" type="cite">
      <pre wrap="">

Thanks for catching this, a new patch is attached.
</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>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Thank you,
Dmitri Pal

Sr. Engineering Manager for IdM portfolio
Red Hat Inc.


-------------------------------
Looking to carve out IT costs?
<a class="moz-txt-link-abbreviated" href="http://www.redhat.com/carveoutcosts/">www.redhat.com/carveoutcosts/</a>


</pre>
  </body>
</html>