diff options
| author | Pavel Březina <pbrezina@redhat.com> | 2013-11-07 11:09:48 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-12-19 20:10:23 +0100 |
| commit | ccb2fe6ee1397b3c3d413d6c546cb88701958de3 (patch) | |
| tree | 399d773d1dafd6f6798e52af5c99a56db8b414f8 /src | |
| parent | 6b44bc4465b954183f8a52fbb05da6b63b17f0d1 (diff) | |
| download | sssd-ccb2fe6ee1397b3c3d413d6c546cb88701958de3.tar.gz sssd-ccb2fe6ee1397b3c3d413d6c546cb88701958de3.tar.xz sssd-ccb2fe6ee1397b3c3d413d6c546cb88701958de3.zip | |
free idmapped binary SIDs correctly
Resolves:
https://fedorahosted.org/sssd/ticket/2133
Diffstat (limited to 'src')
| -rw-r--r-- | src/responder/nss/nsssrv_cmd.c | 2 | ||||
| -rw-r--r-- | src/tests/sss_idmap-tests.c | 10 |
2 files changed, 6 insertions, 6 deletions
diff --git a/src/responder/nss/nsssrv_cmd.c b/src/responder/nss/nsssrv_cmd.c index 07f311880..550017c0e 100644 --- a/src/responder/nss/nsssrv_cmd.c +++ b/src/responder/nss/nsssrv_cmd.c @@ -4341,7 +4341,7 @@ static int nss_cmd_getbysid(enum sss_cli_command cmd, struct cli_ctx *cctx) /* If the body isn't a SID, fail */ err = sss_idmap_sid_to_bin_sid(nctx->idmap_ctx, sid_str, &bin_sid, &bin_sid_length); - talloc_free(bin_sid); + sss_idmap_free_bin_sid(nctx->idmap_ctx, bin_sid); if (err != IDMAP_SUCCESS) { DEBUG(SSSDBG_OP_FAILURE, ("sss_idmap_sid_to_bin_sid failed for [%s].\n", body)); diff --git a/src/tests/sss_idmap-tests.c b/src/tests/sss_idmap-tests.c index 888be8224..f5ec68383 100644 --- a/src/tests/sss_idmap-tests.c +++ b/src/tests/sss_idmap-tests.c @@ -242,7 +242,7 @@ START_TEST(idmap_test_bin_sid2uid) "sss_idmap_bin_sid_to_unix returned wrong id, " "got [%d], expected [%d].", id, 1000 + IDMAP_RANGE_MIN); - talloc_free(bin_sid); + sss_idmap_free_bin_sid(idmap_ctx, bin_sid); } END_TEST @@ -331,7 +331,7 @@ START_TEST(idmap_test_uid2bin_sid) "expected [%s], got [%s].", "S-1-5-21-1-2-3-1000", sid); sss_idmap_free_sid(idmap_ctx, sid); - talloc_free(bin_sid); + sss_idmap_free_bin_sid(idmap_ctx, bin_sid); } END_TEST @@ -359,7 +359,7 @@ START_TEST(idmap_test_bin_sid2dom_sid) "Binary SIDs do not match."); sss_idmap_free_dom_sid(idmap_ctx, dom_sid); - talloc_free(new_bin_sid); + sss_idmap_free_bin_sid(idmap_ctx, new_bin_sid); } END_TEST @@ -437,7 +437,7 @@ START_TEST(idmap_test_sid2bin_sid) fail_unless(memcmp(bin_sid, test_bin_sid, test_bin_sid_length) == 0, "Binary SIDs do not match"); - talloc_free(bin_sid); + sss_idmap_free_bin_sid(idmap_ctx, bin_sid); } END_TEST @@ -496,7 +496,7 @@ START_TEST(idmap_test_smb_sid2bin_sid) fail_unless(memcmp(bin_sid, test_bin_sid, test_bin_sid_length) == 0, "Binary SIDs do not match."); - talloc_free(bin_sid); + sss_idmap_free_bin_sid(idmap_ctx, bin_sid); } END_TEST |