diff options
| author | Nalin Dahyabhai <nalin@dahyabhai.net> | 2012-01-11 19:30:06 -0500 |
|---|---|---|
| committer | Nalin Dahyabhai <nalin@dahyabhai.net> | 2012-01-11 19:30:06 -0500 |
| commit | 914642f65fcf35e48e77bd3c8ce377539af9045a (patch) | |
| tree | 32e9179737cdefc7a3c9f68220399b87f6bbd706 /src/back-sch.c | |
| parent | 9743d79ec585494febdd2029b9094e948f09ac67 (diff) | |
| download | slapi-nis-914642f65fcf35e48e77bd3c8ce377539af9045a.tar.gz slapi-nis-914642f65fcf35e48e77bd3c8ce377539af9045a.tar.xz slapi-nis-914642f65fcf35e48e77bd3c8ce377539af9045a.zip | |
- avoid using pblocks for more than one internal operation - the server
allocates internal state each time but doesn't clean up any that's
aready there if you reuse the block
- correctly free values we use when constructing compat entries
Diffstat (limited to 'src/back-sch.c')
| -rw-r--r-- | src/back-sch.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/back-sch.c b/src/back-sch.c index d114aed..3a91a74 100644 --- a/src/back-sch.c +++ b/src/back-sch.c @@ -403,7 +403,7 @@ backend_set_entry(Slapi_Entry *e, struct backend_set_data *data) } /* Clean up the values. */ for (j = 0; j < k; j++) { - slapi_value_free(&value[k]); + slapi_value_free(&value[j]); } free(value); } @@ -666,6 +666,7 @@ backend_get_set_config(struct plugin_state *state, NULL, backend_get_set_config_entry_if_matching_cb, NULL); + slapi_pblock_destroy(pb); /* Return the results. */ *bases = cbdata.bases; @@ -678,7 +679,6 @@ backend_get_set_config(struct plugin_state *state, slapi_sdn_free(&setrdn); } slapi_sdn_free(&groupdn); - slapi_pblock_destroy(pb); } /* Given an entry, return the filter which will match a container entry beneath |