diff options
| author | Sumit Bose <sbose@redhat.com> | 2015-11-05 08:53:33 +0100 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2015-11-05 08:57:05 +0100 |
| commit | 8cfc3d5aeea21a3b8d1801aeef4866a4d3e7add0 (patch) | |
| tree | 1c984f235599e298ee9083c53f9ec41a3dbf43b2 | |
| parent | 2d5d7761ef2b0d43c39dadf877b87aae19231036 (diff) | |
| download | sssd-8cfc3d5aeea21a3b8d1801aeef4866a4d3e7add0.tar.gz sssd-8cfc3d5aeea21a3b8d1801aeef4866a4d3e7add0.tar.xz sssd-8cfc3d5aeea21a3b8d1801aeef4866a4d3e7add0.zip | |
pam-srv-tests: Change service name
It would be better to not use 'ssh' or 'sshd' here at all but something like
'pam_test_service' to indicate that it is a generic name.
Because a default value should not lead to a code path which handles a
special case. The general PAM responder test should not run through the
'sshd' case in pam_reply() only if the service is set explicitly to
'sshd' this features should be tests.
Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>
| -rw-r--r-- | src/tests/cmocka/test_pam_srv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/tests/cmocka/test_pam_srv.c b/src/tests/cmocka/test_pam_srv.c index dbdc4ae08..9f0cdcdb0 100644 --- a/src/tests/cmocka/test_pam_srv.c +++ b/src/tests/cmocka/test_pam_srv.c @@ -429,7 +429,7 @@ static void mock_input_pam(TALLOC_CTX *mem_ctx, const char *name, } } - pi.pam_service = "ssh"; + pi.pam_service = "pam_test_service"; pi.pam_service_size = strlen(pi.pam_service) + 1; pi.pam_tty = "/dev/tty"; pi.pam_tty_size = strlen(pi.pam_tty) + 1; |