diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2016-04-08 11:47:44 +0200 |
---|---|---|
committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-04-13 15:44:23 +0200 |
commit | 27a0be2bb6f21f66527e0edea4ed2cb4b5cafa53 (patch) | |
tree | 476f490cbf5a9218b65618ad374283911f6059d1 /src/ldb_modules | |
parent | c07fb3f111b4dc2780fa4e1408ea04cd36e95a4d (diff) | |
download | sssd-27a0be2bb6f21f66527e0edea4ed2cb4b5cafa53.tar.gz sssd-27a0be2bb6f21f66527e0edea4ed2cb4b5cafa53.tar.xz sssd-27a0be2bb6f21f66527e0edea4ed2cb4b5cafa53.zip |
memberof: Don't allocate on NULL when deleting memberUids
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Diffstat (limited to 'src/ldb_modules')
-rw-r--r-- | src/ldb_modules/memberof.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/ldb_modules/memberof.c b/src/ldb_modules/memberof.c index 0d909bfe8..5fda9deb8 100644 --- a/src/ldb_modules/memberof.c +++ b/src/ldb_modules/memberof.c @@ -2471,7 +2471,7 @@ static int mbof_del_fill_muop(struct mbof_del_ctx *del_ctx, for (i = 0; i < el->num_values; i++) { struct ldb_dn *valdn; - valdn = ldb_dn_from_ldb_val(del_ctx->muops, + valdn = ldb_dn_from_ldb_val(del_ctx, ldb_module_get_ctx(del_ctx->ctx->module), &el->values[i]); if (!valdn || !ldb_dn_validate(valdn)) { @@ -2489,6 +2489,7 @@ static int mbof_del_fill_muop(struct mbof_del_ctx *del_ctx, if (ret != LDB_SUCCESS) { return ret; } + talloc_steal(del_ctx->muops, valdn); } return LDB_SUCCESS; |