URL: https://github.com/SSSD/sssd/pull/5643
Title: #5643: SECRETS: Resolve mkey path correctly
pbrezina commented:
"""
@justin-stephenson Tests fails:
```
/shared/sssd/src/tests/cmocka/test_kcm_renewals.c: In function
'test_kcm_renewals_tgt':
/shared/sssd/src/tests/cmocka/test_kcm_renewals.c:202:34: warning: passing
argument 4 of 'sss_sec_init_with_path' from incompatible pointer type
[-Wincompatible-pointer-types]
202 | &secdb->sctx);
| ^~~~~~~~~~~~
| |
| struct sss_sec_ctx **
In file included from /shared/sssd/src/responder/kcm/kcmsrv_ccache_secdb.c:28,
from /shared/sssd/src/tests/cmocka/test_kcm_renewals.c:35:
/shared/sssd/src/util/secrets/secrets.h:86:44: note: expected 'const char *'
but argument is of type 'struct sss_sec_ctx **'
86 | const char *mkeypath,
| ~~~~~~~~~~~~^~~~~~~~
/shared/sssd/src/tests/cmocka/test_kcm_renewals.c:201:11: error: too few
arguments to function 'sss_sec_init_with_path'
201 | ret = sss_sec_init_with_path(test_ctx->ccdb, NULL,
TEST_DB_FULL_PATH,
| ^~~~~~~~~~~~~~~~~~~~~~
In file included from /shared/sssd/src/responder/kcm/kcmsrv_ccache_secdb.c:28,
from /shared/sssd/src/tests/cmocka/test_kcm_renewals.c:35:
/shared/sssd/src/util/secrets/secrets.h:83:9: note: declared here
83 | errno_t sss_sec_init_with_path(TALLOC_CTX *mem_ctx,
| ^~~~~~~~~~~~~~~~~~~~~~
make[3]: *** [Makefile:40133:
src/tests/cmocka/test_kcm_renewals-test_kcm_renewals.o] Error 1
```
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5643#issuecomment-844237509
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct:
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives:
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it:
https://pagure.io/fedora-infrastructure
