File clvmd-fix-debugging-level-set-in-clvmd_set_debu... of Package lvm2.1005

Overview Repositories Revisions Requests Users Attributes Meta

File clvmd-fix-debugging-level-set-in-clvmd_set_debug-function.patch of Package lvm2.1005

From 13fe333b54c2dedf91af2f663d6ac9bd883c3b87 Mon Sep 17 00:00:00 2001
From: Zdenek Kabelac <zkabelac@redhat.com>
Date: Fri, 19 Oct 2012 15:18:19 +0200
Subject: clvmd: fix parsing of -d argument
Git-repo: https://git.fedorahosted.org/git/lvm2.git
Git-commit: 13fe333b54c2dedf91af2f663d6ac9bd883c3b87
References: bnc#785467

clvmd -d option parsing was not working properly.

clvmd -d 2   (with space) has been ignored because of
'::' used in getopt string, and as failsafe it's been used '1'.

Later this debug_arg has been ignored and debug_opt was used
instead which happend to have value '1'.

Submitted-by: Robert Milasan <rmilasan@suse.com>
Reported-by: Robert Milasan  <rmilasan@suse.com>
---

daemons/clvmd/clvmd.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/daemons/clvmd/clvmd.c b/daemons/clvmd/clvmd.c
index ac465d9..eb0bffd 100644
--- a/daemons/clvmd/clvmd.c
+++ b/daemons/clvmd/clvmd.c
@@ -364,7 +364,7 @@ int main(int argc, char *argv[])
 	/* Deal with command-line arguments */
 	opterr = 0;
 	optind = 0;
-	while ((opt = getopt_long(argc, argv, "vVhfd::t:RST:CI:E:",
+	while ((opt = getopt_long(argc, argv, "vVhfd:t:RST:CI:E:",
 				  longopts, NULL)) != -1) {
 		switch (opt) {
 		case 'h':
@@ -451,7 +451,7 @@ int main(int argc, char *argv[])
 		return debug_clvmd(debug_arg, clusterwide_opt)==1?0:1;
 	}
 
-	clvmd_set_debug(debug_opt);
+	clvmd_set_debug(debug_arg);
 
 	/* Fork into the background (unless requested not to) */
 	if (!foreground_mode)

Places

File clvmd-fix-debugging-level-set-in-clvmd_set_debug-function.patch of Package lvm2.1005

Places