summaryrefslogtreecommitdiff
path: root/auth
diff options
context:
space:
mode:
authorAlexander Bokovoy <ab@samba.org>2020-11-04 19:21:20 +0200
committerAlexander Bokovoy <ab@samba.org>2020-11-05 06:30:31 +0000
commit5d80b179a1ae43325dde821fc6a869b6c1a1eeea (patch)
tree3770b2a2d63c512bf9321529f4d0f7137fcc375b /auth
parent49efe0ca0b07396764f78434989165f941cf5f91 (diff)
downloadsamba-5d80b179a1ae43325dde821fc6a869b6c1a1eeea.tar.gz
Revert "cli_credentials: add a helper to parse user or group names"
This reverts commit 00f4262ed0b22f6e333e5a29c5590b62c783905c.
Diffstat (limited to 'auth')
-rw-r--r--auth/credentials/credentials.c18
-rw-r--r--auth/credentials/credentials.h1
-rw-r--r--auth/credentials/tests/test_creds.c56
3 files changed, 0 insertions, 75 deletions
diff --git a/auth/credentials/credentials.c b/auth/credentials/credentials.c
index 21c170c7ad9..1bdd6f15a09 100644
--- a/auth/credentials/credentials.c
+++ b/auth/credentials/credentials.c
@@ -836,24 +836,6 @@ _PUBLIC_ void cli_credentials_parse_string(struct cli_credentials *credentials,
cli_credentials_set_password(credentials, p+1, obtained);
}
- cli_credentials_parse_name(credentials, uname, obtained);
-}
-
-/**
- * Given a string, parse it into a domain, username and realm fields
- *
- * The format accepted is [domain\\]user or user[@realm]
- *
- * @param credentials Credentials structure on which to set the components
- * @param data the string containing the username, prefixed or suffixed with domain or realm
- * @param obtained This enum describes how 'specified' this credential name is.
- */
-
-_PUBLIC_ void cli_credentials_parse_name(struct cli_credentials *credentials, const char *data, enum credentials_obtained obtained)
-{
- char *uname, *p;
-
- uname = talloc_strdup(credentials, data);
if ((p = strchr_m(uname,'@'))) {
/*
* We also need to set username and domain
diff --git a/auth/credentials/credentials.h b/auth/credentials/credentials.h
index 7c7120b9f55..f468b8558dd 100644
--- a/auth/credentials/credentials.h
+++ b/auth/credentials/credentials.h
@@ -155,7 +155,6 @@ bool cli_credentials_set_password(struct cli_credentials *cred,
enum credentials_obtained obtained);
struct cli_credentials *cli_credentials_init_anon(TALLOC_CTX *mem_ctx);
void cli_credentials_parse_string(struct cli_credentials *credentials, const char *data, enum credentials_obtained obtained);
-void cli_credentials_parse_name(struct cli_credentials *credentials, const char *data, enum credentials_obtained obtained);
struct samr_Password *cli_credentials_get_nt_hash(struct cli_credentials *cred,
TALLOC_CTX *mem_ctx);
struct samr_Password *cli_credentials_get_old_nt_hash(struct cli_credentials *cred,
diff --git a/auth/credentials/tests/test_creds.c b/auth/credentials/tests/test_creds.c
index ac5bc6d9e04..d2d3d30d73d 100644
--- a/auth/credentials/tests/test_creds.c
+++ b/auth/credentials/tests/test_creds.c
@@ -197,61 +197,6 @@ static void torture_creds_parse_string(void **state)
assert_int_equal(creds->password_obtained, CRED_SPECIFIED);
}
-static void _parse_name_as_lookup_name(TALLOC_CTX *mem_ctx,
- const char *full_name,
- const char *expected_name,
- const char *expected_domain,
- const char *expected_realm)
-{
- struct cli_credentials *creds = NULL;
-
- creds = cli_credentials_init(mem_ctx);
- assert_non_null(creds);
-
- cli_credentials_parse_name(creds, full_name, CRED_SPECIFIED);
-
- if (expected_name == NULL) {
- assert_null(cli_credentials_get_username(creds));
- } else {
- assert_string_equal(cli_credentials_get_username(creds), expected_name);
- }
-
- if (expected_domain == NULL) {
- assert_null(cli_credentials_get_domain(creds));
- } else {
- assert_string_equal(cli_credentials_get_domain(creds), expected_domain);
- }
-
- if (expected_realm == NULL) {
- assert_null(cli_credentials_get_realm(creds));
- } else {
- assert_string_equal(cli_credentials_get_realm(creds), expected_realm);
- }
-
- TALLOC_FREE(creds);
-
-}
-
-static void torture_creds_parse_name(void **state)
-{
- TALLOC_CTX *mem_ctx = *state;
-
- _parse_name_as_lookup_name(mem_ctx, "XXL\\",
- "", "XXL", NULL);
-
- _parse_name_as_lookup_name(mem_ctx, "XXL\\wurst",
- "wurst", "XXL", NULL);
-
- _parse_name_as_lookup_name(mem_ctx, "wurst@brot.realm",
- "wurst", "", "BROT.REALM");
-
- _parse_name_as_lookup_name(mem_ctx, "wur%t",
- "wur%t", NULL, NULL);
-
- _parse_name_as_lookup_name(mem_ctx, "wurst",
- "wurst", NULL, NULL);
-}
-
int main(int argc, char *argv[])
{
int rc;
@@ -261,7 +206,6 @@ int main(int argc, char *argv[])
cmocka_unit_test(torture_creds_guess),
cmocka_unit_test(torture_creds_anon_guess),
cmocka_unit_test(torture_creds_parse_string),
- cmocka_unit_test(torture_creds_parse_name),
};
if (argc == 2) {