From 8414d150cee62ba0554cfd645956a88dba02a7eb Mon Sep 17 00:00:00 2001 From: Steve Dickson Date: Fri, 7 Aug 2009 14:34:42 -0400 Subject: [PATCH] Now that only the Section names are case-insensitive the mount code has to make sure the the mount options given to the kernel are in the correct case. Fixed a couple of warnings on #ifndefs Signed-off-by: Steve Dickson --- support/include/conffile.h | 10 ++++++++++ support/nfs/conffile.c | 12 ------------ utils/mount/configfile.c | 9 ++++++--- 3 files changed, 16 insertions(+), 15 deletions(-) diff --git a/support/include/conffile.h b/support/include/conffile.h index b263581..132a149 100644 --- a/support/include/conffile.h +++ b/support/include/conffile.h @@ -64,4 +64,14 @@ extern int conf_remove(int, char *, char *); extern int conf_remove_section(int, char *); extern void conf_report(void); +/* + * Convert letter from upper case to lower case + */ +static inline void upper2lower(char *str) +{ + char c; + + while ((c = tolower(*str))) + *str++ = c; +} #endif /* _CONFFILE_H_ */ diff --git a/support/nfs/conffile.c b/support/nfs/conffile.c index 97dc88a..b19b739 100644 --- a/support/nfs/conffile.c +++ b/support/nfs/conffile.c @@ -119,18 +119,6 @@ conf_hash(char *s) return hash; } -/* - * Convert letter from upper case to lower case - */ -static inline void upper2lower(char *str) -{ - char *ptr = str; - - while (*ptr) - *ptr++ = tolower(*ptr); -} - - /* * Insert a tag-value combination from LINE (the equal sign is at POS) */ diff --git a/utils/mount/configfile.c b/utils/mount/configfile.c index cad8c52..e347b0e 100644 --- a/utils/mount/configfile.c +++ b/utils/mount/configfile.c @@ -37,11 +37,11 @@ #define NFSMOUNT_GLOBAL_OPTS "NFSMount_Global_Options" #endif -#ifndef NFSMOUNT_MOUNTPOINT "MountPoint" +#ifndef NFSMOUNT_MOUNTPOINT #define NFSMOUNT_MOUNTPOINT "MountPoint" #endif -#ifndef NFSMOUNT_SERVER "Server" +#ifndef NFSMOUNT_SERVER #define NFSMOUNT_SERVER "Server" #endif @@ -83,6 +83,9 @@ char *mountopts_alias(char *opt, int *argtype) *argtype = mnt_alias_tab[i].argtype; return mnt_alias_tab[i].opt; } + /* Make option names case-insensitive */ + upper2lower(opt); + return opt; } /* @@ -202,7 +205,7 @@ conf_parse_mntopts(char *section, char *arg, char *opts) * Do not overwrite options if already exists */ snprintf(buf, BUFSIZ, "%s=", node->field); - if (opts && strstr(opts, buf) != NULL) + if (opts && strcasestr(opts, buf) != NULL) continue; if (lookup_entry(node->field) != NULL) continue; -- 2.39.2