From: Mike Frysinger <vapier@gentoo.org>
Date: Wed, 7 Jun 2006 14:25:22 +0000 (-0000)
Subject: Tito writes: this patch fixes two potential bugs with the -i -I flags that could... 
X-Git-Tag: 1_2_0~201
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=ea93f8a397772e515fb54f6051836218112abf3f;p=oweals%2Fbusybox.git

Tito writes: this patch fixes two potential bugs with the -i -I flags that could be reset during option parsing
---

diff --git a/miscutils/hdparm.c b/miscutils/hdparm.c
index 5ac7bdcb7..5703eceb5 100644
--- a/miscutils/hdparm.c
+++ b/miscutils/hdparm.c
@@ -2122,8 +2122,8 @@ int hdparm_main(int argc, char **argv)
 	while ((c = getopt(argc, argv, hdparm_options)) >= 0) {
 		flagcount++;
 		if (c == 'h') bb_show_usage(); /* EXIT */
-		USE_FEATURE_HDPARM_GET_IDENTITY(get_IDentity = (c == 'I'));
-		USE_FEATURE_HDPARM_GET_IDENTITY(get_identity = (c == 'i'));
+		USE_FEATURE_HDPARM_GET_IDENTITY(get_IDentity |= (c == 'I'));
+		USE_FEATURE_HDPARM_GET_IDENTITY(get_identity |= (c == 'i'));
 		get_geom |= (c == 'g');
 		do_flush |= (c == 'f');
 		if (c == 'u') parse_opts(&get_unmask, &set_unmask, &unmask, 0, 1);