#1703 Wrong variable check in the memberof plugin
Closed: Fixed None Opened 8 years ago by jhrozek.

Probably a copy-and-paste error:

3016    mod_ctx->igh = talloc_zero(mod_ctx, struct mbof_mod_del_op);
Dereferencing "mod_ctx" before a null check.
3017    if (mod_ctx == NULL) {
3018        return LDB_ERR_OPERATIONS_ERROR;
3019    }

We should be checking for mod_ctx->igh

Fields changed

coverity: => 13119
keywords: => Coverity

Fields changed

owner: somebody => jhrozek
status: new => assigned

Fields changed

patch: 0 => 1

milestone: NEEDS_TRIAGE => SSSD 1.9.4
resolution: => fixed
status: assigned => closed

Coverity bug. No clone needed.

rhbz: => 0

Metadata Update from @jhrozek:
- Issue assigned to jhrozek
- Issue set to the milestone: SSSD 1.9.4

4 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/2745

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.