[SSSD] [sssd PR#286][edited] I found two Coverity warnings in the current master while looking for another issue.

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/286
Author: jhrozek
 Title: #286: I found two Coverity warnings in the current master while looking for another issue.
Action: edited

Changed field: body
Original value:
"""
IPA: Return from function after marking a request as finished

Fixes:
    Error: UNINIT (CWE-457):
    sssd-1.15.3/src/providers/ipa/ipa_s2n_exop.c:782: var_decl: Declaring variable "ret" without initializer.
    sssd-1.15.3/src/providers/ipa/ipa_s2n_exop.c:1001: uninit_use: Using uninitialized value "ret".
    #  999|                   name = NULL;
    # 1000|                   fq_name = NULL;
    # 1001|->                 if (ret != EOK) {
    # 1002|                       DEBUG(SSSDBG_OP_FAILURE, "add_to_name_list failed.\n");
    # 1003|                       goto done;

IPA: Avoid using uninitialized ret value when skipping entries from the joined domain

Fixes:
   Error: FORWARD_NULL (CWE-476):
   sssd-1.15.3/src/providers/ipa/ipa_s2n_exop.c:2659: var_compare_op: Comparing "state->attrs" to null implies that "state->attrs" might be null.
   sssd-1.15.3/src/providers/ipa/ipa_s2n_exop.c:2668: var_deref_op: Dereferencing null pointer "state->attrs".
   # 2666|       }
   # 2667|   
   # 2668|->     ret = sysdb_attrs_get_string(state->attrs->sysdb_attrs, SYSDB_SID_STR,
   # 2669|                                    &sid_str);
   # 2670|       if (ret == ENOENT) {
"""