Removed parameter -p and added parameter -d to allow to drop privileges.

Default is to NOT drop the privileges as is causes permission problems
on some systems.

1 files changed, 7 insertions, 7 deletions

diff --git a/src/main.c b/src/main.c
index f672231..99d2a5c 100644
--- a/src/main.c
+++ b/src/main.c
@@ -57,7 +57,7 @@ static int fsock = -1;
 static int verbose = DEBUG_LEVEL;
 static int foreground = 0;
 static int exit_on_no_devices = 0;
-static int preserve_privileges = 0;
+static int drop_privileges = 0;
 
 struct device_info {
 	uint32_t device_id;
@@ -1006,7 +1006,7 @@ static void usage()
 	printf("\t-v|--verbose              be verbose (use twice or more to increase)\n");
 	printf("\t-f|--foreground           do not daemonize\n");
 	printf("\t-e|--exit-on-no-devices   exit if no device is attached\n");
-	printf("\t-p|--preserve-privileges  do not drop privileges\n");
+	printf("\t-d|--drop-privileges      drop privileges after startup\n");
 	printf("\n");
 }
 
@@ -1017,13 +1017,13 @@ static void parse_opts(int argc, char **argv)
 		{"foreground", 0, NULL, 'f'},
 		{"verbose", 0, NULL, 'v'},
 		{"exit-on-no-devices", 0, NULL, 'e'},
-		{"preserve-privileges", 0, NULL, 'p'},
+		{"drop-privileges", 0, NULL, 'd'},
 		{NULL, 0, NULL, 0}
 	};
 	int c;
 
 	while (1) {
-		c = getopt_long(argc, argv, "hfvep", longopts, (int *) 0);
+		c = getopt_long(argc, argv, "hfved", longopts, (int *) 0);
 		if (c == -1) {
 			break;
 		}
@@ -1041,8 +1041,8 @@ static void parse_opts(int argc, char **argv)
 		case 'e':
 			exit_on_no_devices = 1;
 			break;
-		case 'p':
-			preserve_privileges = 1;
+		case 'd':
+			drop_privileges = 1;
 			break;
 		default:
 			usage();
@@ -1200,7 +1200,7 @@ int main(int argc, char **argv)
 		}
 	}
 	// drop elevated privileges
-	if (!preserve_privileges && (getuid() == 0 || geteuid() == 0)) {
+	if (drop_privileges && (getuid() == 0 || geteuid() == 0)) {
 		struct passwd *pw = getpwnam("nobody");
 		if (pw) {
 			setuid(pw->pw_uid);