Ticket has been cloned to Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=842437

Sending fix out for review...

It's not that simple. The problem is that you cannot just call slapi_mods_free(&smods) in every case.
{{{
slapi_pblock_get(pb, SLAPI_MODIFY_MODS, &mods);
smods = slapi_mods_new();
slapi_mods_init_passin(smods, mods);
}}}
once this happens, smods owns mods. This means when you call slapi_mods_free(&smods) it will free the mods as well, since it owns that memory.

I have new patch (attached) but I can't verify it without a testcase. I would think its ok, as I just removed the allocation all together, and used a local variable.

Could still leak memory:
{{{
if (LDAP_CHANGETYPE_ADD == modtype) {
ret = _dna_pre_op_add(pb, test_e);
} else {
- ret = _dna_pre_op_modify(pb, test_e, smods);
+ ret = _dna_pre_op_modify(pb, test_e, &smods);
}
if (ret) {
goto bail;
}}}
If ret is set and it does the goto bail, if smods has been changed, those changes will be leaked. Also, it's not a good idea to use the structure directly - better to stick with the slapi API.

Ok, here is patch number 3...

#3
0001-Ticket-407-memory-leak-in-dna-plugin.2.patch

Thanks for the review Rich!

git merge ticket407
Updating 657efc6..9229db2
Fast-forward
ldap/servers/plugins/dna/dna.c | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)

git push origin master
Counting objects: 13, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (7/7), done.
Writing objects: 100% (7/7), 854 bytes, done.
Total 7 (delta 5), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
657efc6..9229db2 master -> master

Fixes crash
0001-Ticket-407-dna-memory-leak.patch

[mareynol@localhost ds]$ git merge ticket407
Updating 52e1507..60316fc
Fast-forward
ldap/servers/plugins/dna/dna.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

[mareynol@localhost ds]$ git push origin master
Counting objects: 13, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (7/7), done.
Writing objects: 100% (7/7), 682 bytes, done.
Total 7 (delta 5), reused 0 (delta 0)
To ssh://git.fedorahosted.org/git/389/ds.git
52e1507..60316fc master -> master

Added initial screened field value.

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

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/407

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.