[PATCH] ksmbd-tools: change default db file name to users.db

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This commit changes the default file name for the users database from
/etc/ksmbd/ksmbdpwd.db to /etc/ksmbd/users.db, which is more reasonable
and makes more sense for end users.

Also rename some variables and functions that dealt with this file.

Signed-off-by: Enzo Matsumiya <ematsumiya@xxxxxxx>
---
 adduser/adduser.c       | 20 ++++++++++----------
 adduser/user_admin.c    | 20 ++++++++++----------
 include/config_parser.h |  2 +-
 include/ksmbdtools.h    |  4 ++--
 lib/config_parser.c     |  4 ++--
 mountd/ipc.c            |  6 +++---
 mountd/mountd.c         | 14 +++++++-------
 7 files changed, 35 insertions(+), 35 deletions(-)

diff --git a/adduser/adduser.c b/adduser/adduser.c
index 54774d3d6e15..e4d17460b4c9 100644
--- a/adduser/adduser.c
+++ b/adduser/adduser.c
@@ -56,15 +56,15 @@ static void show_version(void)
 	exit(EXIT_FAILURE);
 }
 
-static int parse_configs(char *pwddb)
+static int parse_configs(char *db)
 {
 	int ret;
 
-	ret = test_file_access(pwddb);
+	ret = test_file_access(db);
 	if (ret)
 		return ret;
 
-	ret = cp_parse_pwddb(pwddb);
+	ret = cp_parse_db(db);
 	if (ret)
 		return ret;
 	return 0;
@@ -97,7 +97,7 @@ static int sanity_check_user_name_simple(char *uname)
 int main(int argc, char *argv[])
 {
 	int ret = EXIT_FAILURE;
-	char *pwddb = PATH_PWDDB;
+	char *db = PATH_USERS_DB;
 	int c, cmd = 0;
 
 	set_logger_app_name("smbuseradd");
@@ -121,7 +121,7 @@ int main(int argc, char *argv[])
 			arg_password = g_strdup(optarg);
 			break;
 		case 'i':
-			pwddb = g_strdup(optarg);
+			db = g_strdup(optarg);
 			break;
 		case 'V':
 			show_version();
@@ -139,7 +139,7 @@ int main(int argc, char *argv[])
 		goto out;
 	}
 
-	if (!pwddb) {
+	if (!db) {
 		pr_err("Out of memory\n");
 		goto out;
 	}
@@ -156,18 +156,18 @@ int main(int argc, char *argv[])
 		goto out;
 	}
 
-	ret = parse_configs(pwddb);
+	ret = parse_configs(db);
 	if (ret) {
 		pr_err("Unable to parse configuration files\n");
 		goto out;
 	}
 
 	if (cmd == COMMAND_ADD_USER)
-		ret = command_add_user(pwddb, arg_account, arg_password);
+		ret = command_add_user(db, arg_account, arg_password);
 	if (cmd == COMMAND_DEL_USER)
-		ret = command_del_user(pwddb, arg_account);
+		ret = command_del_user(db, arg_account);
 	if (cmd == COMMAND_UPDATE_USER)
-		ret = command_update_user(pwddb, arg_account, arg_password);
+		ret = command_update_user(db, arg_account, arg_password);
 
 	/*
 	 * We support only ADD_USER command at this moment
diff --git a/adduser/user_admin.c b/adduser/user_admin.c
index 7ea1d43c3540..df1315292bac 100644
--- a/adduser/user_admin.c
+++ b/adduser/user_admin.c
@@ -31,16 +31,16 @@ static char *arg_password = NULL;
 static int conf_fd = -1;
 static char wbuf[2 * MAX_NT_PWD_LEN + 2 * KSMBD_REQ_MAX_ACCOUNT_NAME_SZ];
 
-static int __opendb_file(char *pwddb)
+static int open_db_file(char *db)
 {
-	conf_fd = open(pwddb, O_WRONLY);
+	conf_fd = open(db, O_WRONLY);
 	if (conf_fd == -1) {
-		pr_err("%s %s\n", strerr(errno), pwddb);
+		pr_err("%s %s\n", strerr(errno), db);
 		return -EINVAL;
 	}
 
 	if (ftruncate(conf_fd, 0)) {
-		pr_err("%s %s\n", strerr(errno), pwddb);
+		pr_err("%s %s\n", strerr(errno), db);
 		close(conf_fd);
 		return -EINVAL;
 	}
@@ -295,7 +295,7 @@ conflict:
 	*abort_del_user = 1;
 }
 
-int command_add_user(char *pwddb, char *account, char *password)
+int command_add_user(char *db, char *account, char *password)
 {
 	struct ksmbd_user *user;
 	char *pswd;
@@ -323,7 +323,7 @@ int command_add_user(char *pwddb, char *account, char *password)
 	}
 
 	pr_info("User '%s' added\n", arg_account);
-	if (__opendb_file(pwddb))
+	if (open_db_file(db))
 		return -EINVAL;
 
 	for_each_ksmbd_user(write_user_cb, NULL);
@@ -331,7 +331,7 @@ int command_add_user(char *pwddb, char *account, char *password)
 	return 0;
 }
 
-int command_update_user(char *pwddb, char *account, char *password)
+int command_update_user(char *db, char *account, char *password)
 {
 	struct ksmbd_user *user;
 	char *pswd;
@@ -362,7 +362,7 @@ int command_update_user(char *pwddb, char *account, char *password)
 	put_ksmbd_user(user);
 	free(pswd);
 
-	if (__opendb_file(pwddb))
+	if (open_db_file(db))
 		return -EINVAL;
 
 	for_each_ksmbd_user(write_user_cb, NULL);
@@ -370,7 +370,7 @@ int command_update_user(char *pwddb, char *account, char *password)
 	return 0;
 }
 
-int command_del_user(char *pwddb, char *account)
+int command_del_user(char *db, char *account)
 {
 	int abort_del_user = 0;
 
@@ -388,7 +388,7 @@ int command_del_user(char *pwddb, char *account)
 		return -EINVAL;
 	}
 
-	if (__opendb_file(pwddb))
+	if (open_db_file(db))
 		return -EINVAL;
 
 	for_each_ksmbd_user(write_remove_user_cb, NULL);
diff --git a/include/config_parser.h b/include/config_parser.h
index c051f487c319..0aefc3b4d5c7 100644
--- a/include/config_parser.h
+++ b/include/config_parser.h
@@ -26,7 +26,7 @@ int cp_parse_external_smbconf_group(char *name, char *opts);
 int cp_smbconfig_hash_create(const char *smbconf);
 void cp_smbconfig_destroy(void);
 
-int cp_parse_pwddb(const char *pwddb);
+int cp_parse_db(const char *db);
 int cp_parse_smbconf(const char *smbconf);
 int cp_parse_reload_smbconf(const char *smbconf);
 int cp_parse_subauth(const char *subauth_path);
diff --git a/include/ksmbdtools.h b/include/ksmbdtools.h
index 5a1236878613..2d55fefdffed 100644
--- a/include/ksmbdtools.h
+++ b/include/ksmbdtools.h
@@ -56,7 +56,7 @@ struct smbconf_global {
 	unsigned int		gen_subauth[3];
 	char			*krb5_keytab_file;
 	char			*krb5_service_name;
-	char			*pwddb;
+	char			*users_db;
 	char			*smbconf;
 };
 
@@ -84,7 +84,7 @@ extern struct smbconf_global global_conf;
 
 #define KSMBD_CONF_FILE_MAX		10000
 
-#define PATH_PWDDB	"/etc/ksmbd/ksmbdpwd.db"
+#define PATH_USERS_DB	"/etc/ksmbd/users.db"
 #define PATH_SMBCONF	"/etc/ksmbd/smb.conf"
 
 #define KSMBD_HEALTH_START		(0)
diff --git a/lib/config_parser.c b/lib/config_parser.c
index ebbe2dd4e69f..da82a95955f3 100644
--- a/lib/config_parser.c
+++ b/lib/config_parser.c
@@ -674,9 +674,9 @@ int cp_parse_smbconf(const char *smbconf)
 				    GROUPS_CALLBACK_STARTUP_INIT);
 }
 
-int cp_parse_pwddb(const char *pwddb)
+int cp_parse_db(const char *db)
 {
-	return __mmap_parse_file(pwddb, usm_add_update_user_from_pwdentry);
+	return __mmap_parse_file(db, usm_add_update_user_from_pwdentry);
 }
 
 int cp_smbconfig_hash_create(const char *smbconf)
diff --git a/mountd/ipc.c b/mountd/ipc.c
index 15c59f5aa850..be2b6a2c3fed 100644
--- a/mountd/ipc.c
+++ b/mountd/ipc.c
@@ -63,13 +63,13 @@ static int generic_event(int type, void *payload, size_t sz)
 	return 0;
 }
 
-static int parse_reload_configs(const char *pwddb, const char *smbconf)
+static int parse_reload_configs(const char *db, const char *smbconf)
 {
 	int ret;
 
 	pr_debug("Reload config\n");
 	usm_remove_all_users();
-	ret = cp_parse_pwddb(pwddb);
+	ret = cp_parse_db(db);
 	if (ret == -ENOENT) {
 		pr_err("User database file does not exist. %s\n",
 		       "Only guest sessions (if permitted) will work.");
@@ -91,7 +91,7 @@ static int handle_generic_event(struct nl_cache_ops *unused,
 				void *arg)
 {
 	if (ksmbd_health_status & KSMBD_SHOULD_RELOAD_CONFIG) {
-		parse_reload_configs(global_conf.pwddb, global_conf.smbconf);
+		parse_reload_configs(global_conf.users_db, global_conf.smbconf);
 		ksmbd_health_status &= ~KSMBD_SHOULD_RELOAD_CONFIG;
 	}
 
diff --git a/mountd/mountd.c b/mountd/mountd.c
index 71a4d985d5a9..2476f62d0355 100644
--- a/mountd/mountd.c
+++ b/mountd/mountd.c
@@ -44,7 +44,7 @@ static void usage(void)
 	fprintf(stderr, "Usage: ksmbd\n");
 	fprintf(stderr, "\t--p=NUM | --port=NUM              TCP port NUM\n");
 	fprintf(stderr, "\t--c=smb.conf | --config=smb.conf  config file\n");
-	fprintf(stderr, "\t--u=pwd.db | --users=pwd.db       Users DB\n");
+	fprintf(stderr, "\t--u=users.db | --users=users.db   Users DB\n");
 	fprintf(stderr, "\t--n | --nodetach                  Don't detach\n");
 	fprintf(stderr, "\t--s | --systemd                   Service mode\n");
 	fprintf(stderr, "\t-V | --version                    Show version\n");
@@ -288,11 +288,11 @@ static int setup_signals(sighandler_t handler)
 	return 0;
 }
 
-static int parse_configs(char *pwddb, char *smbconf)
+static int parse_configs(char *db, char *smbconf)
 {
 	int ret;
 
-	ret = cp_parse_pwddb(pwddb);
+	ret = cp_parse_db(db);
 	if (ret == -ENOENT) {
 		pr_err("User database file does not exist. %s\n",
 			"Only guest sessions (if permitted) will work.");
@@ -392,7 +392,7 @@ static int worker_process_init(void)
 		goto out;
 	}
 
-	ret = parse_configs(global_conf.pwddb, global_conf.smbconf);
+	ret = parse_configs(global_conf.users_db, global_conf.smbconf);
 	if (ret) {
 		pr_err("Failed to parse configuration files\n");
 		goto out;
@@ -562,7 +562,7 @@ int main(int argc, char *argv[])
 
 	set_logger_app_name("ksmbd-manager");
 	memset(&global_conf, 0x00, sizeof(struct smbconf_global));
-	global_conf.pwddb = PATH_PWDDB;
+	global_conf.users_db = PATH_USERS_DB;
 	global_conf.smbconf = PATH_SMBCONF;
 	pr_logger_init(PR_LOGGER_STDIO);
 
@@ -586,7 +586,7 @@ int main(int argc, char *argv[])
 			global_conf.smbconf = g_strdup(optarg);
 			break;
 		case 'u':
-			global_conf.pwddb = g_strdup(optarg);
+			global_conf.users_db = g_strdup(optarg);
 			break;
 		case 'n':
 			if (!optarg)
@@ -611,7 +611,7 @@ int main(int argc, char *argv[])
 		}
 	}
 
-	if (!global_conf.smbconf || !global_conf.pwddb) {
+	if (!global_conf.smbconf || !global_conf.users_db) {
 		pr_err("Out of memory\n");
 		exit(EXIT_FAILURE);
 	}
-- 
2.33.0




[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux