summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSteve French <stevef@smf-t60p.smfdom>2008-08-05 13:15:46 -0500
committerSteve French <stevef@smf-t60p.smfdom>2008-08-05 13:15:46 -0500
commit70dccd1468d0a5de123bfa600e7602648e6d2cc1 (patch)
treec78f27aaea6ca0371e197e5a98d423e106f60625
parent500038e48518895410c48f69d97ce3e0cfb0bc5b (diff)
downloadsamba-70dccd1468d0a5de123bfa600e7602648e6d2cc1.tar.gz
samba-70dccd1468d0a5de123bfa600e7602648e6d2cc1.tar.bz2
samba-70dccd1468d0a5de123bfa600e7602648e6d2cc1.zip
Backing out most of changeset 5222b8db3fb692e5071bfd1b41849a8eb0a17995
(so parsing for domain parameter in mount.cifs matches online help) and rephrasing original code to make it more clear. The check for "domain" was meant to allow for "dom" or "DOM" and the option ("dom") described in the help (e.g. "/sbin/mount.cifs -?") is the shorter ("dom") form. The reason that the string we compare against is larger was to improve readability (we could compare against "dom" but note /* "domain" or "DOMAIN" or "dom" or "DOM" */ but it seemed terser to just show the larger string in the strcmp target. The change to "workgoup" from workg* (anything which begins with "workg" doesn't matter - it is a minor behavior change - but probably few scripts depend on the "alias" for this option). Rework code so that it is clearer what we are comparing against. (This used to be commit 92fad0fc537e75c726d5d6794dd0c4fd61edca2d)
-rw-r--r--source3/client/mount.cifs.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/source3/client/mount.cifs.c b/source3/client/mount.cifs.c
index 72ef9febe0..c7009e306c 100644
--- a/source3/client/mount.cifs.c
+++ b/source3/client/mount.cifs.c
@@ -517,8 +517,11 @@ static int parse_options(char ** optionsp, int * filesys_flags)
printf("CIFS: UNC name too long\n");
return 1;
}
- } else if ((strncmp(data, "domain", 6) == 0)
- || (strncmp(data, "workgroup", 9) == 0)) {
+ } else if ((strncmp(data, "dom" /* domain */, 3) == 0)
+ || (strncmp(data, "workg", 5) == 0)) {
+ /* note this allows for synonyms of "domain"
+ such as "DOM" and "dom" and "workgroup"
+ and "WORKGRP" etc. */
if (!value || !*value) {
printf("CIFS: invalid domain name\n");
return 1; /* needs_arg; */