#506 Potential NULL dereference in ipa_auth.c
Closed: Fixed None Opened 13 years ago by sgallagh.

From Coverity:

At conditional (1): "state == NULL" taking the true branch.
Comparing "state" to null implies that "state" might be null.
  238    if (state == NULL) {
  239        DEBUG(1, ("talloc_zero failed.\n"));
  240        goto fail;
  241    }
...
  278fail:
Dereferencing null variable "state".
  279    state->pd->pam_status = PAM_SYSTEM_ERR;
  280    ipa_auth_reply(be_req, DP_ERR_FATAL, state->pd->pam_status);
  281}

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.2.1

Fields changed

keywords: => coverity

Fields changed

status: new => assigned

Fixed by 13cdd7f

fixedin: => 1.2.1
resolution: => fixed
status: assigned => closed

Fields changed

rhbz: => 0

Metadata Update from @sgallagh:
- Issue assigned to sgallagh
- Issue set to the milestone: SSSD 1.2.1

7 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/1548

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata