diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2014-10-18 22:03:01 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-11-18 20:33:36 +0100 |
commit | 7c5cd2e7711621af9163a41393e88896a91ac33b (patch) | |
tree | 9a1eacfb31cecf8893c51a938e312330a423c9e6 /src/tests/krb5_child-test.c | |
parent | 45aeb924ec3ac448bb8d174a5cc061ed98b147c7 (diff) | |
download | sssd-7c5cd2e7711621af9163a41393e88896a91ac33b.tar.gz sssd-7c5cd2e7711621af9163a41393e88896a91ac33b.tar.xz sssd-7c5cd2e7711621af9163a41393e88896a91ac33b.zip |
KRB5: Move checking for illegal RE to krb5_utils.c
Otherwise we would have to link krb5_child with pcre and transfer the
regex, which would be cumbersome. Check for illegal patterns when
expanding the template instead.
Related:
https://fedorahosted.org/sssd/ticket/2370
Reviewed-by: Sumit Bose <sbose@redhat.com>
Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>
Diffstat (limited to 'src/tests/krb5_child-test.c')
-rw-r--r-- | src/tests/krb5_child-test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/tests/krb5_child-test.c b/src/tests/krb5_child-test.c index 09f23d538..8826a28ed 100644 --- a/src/tests/krb5_child-test.c +++ b/src/tests/krb5_child-test.c @@ -239,6 +239,7 @@ create_dummy_req(TALLOC_CTX *mem_ctx, const char *user, kr->ccname = expand_ccname_template(kr, kr, dp_opt_get_cstring(kr->krb5_ctx->opts, KRB5_CCNAME_TMPL), + kr->krb5_ctx->illegal_path_re, true, true); if (!kr->ccname) goto fail; @@ -254,7 +255,6 @@ create_dummy_req(TALLOC_CTX *mem_ctx, const char *user, kr->ccname, kr->uid, kr->gid); ret = sss_krb5_precreate_ccache(kr->ccname, - kr->krb5_ctx->illegal_path_re, kr->uid, kr->gid); if (ret != EOK) { DEBUG(SSSDBG_OP_FAILURE, "create_ccache_dir failed.\n"); |