From 32057bc15469e6f012841605daa38766497fa196 Mon Sep 17 00:00:00 2001
From: Justin Mayfield <jmayfield@cradlepoint.com>
Date: Wed, 9 May 2012 21:08:30 -0600
Subject: __str2flags fix

I found a minor bug in __str2flags where empty strings or short strings
will match all or many flags respectively.  Basically the test needs to
ensure the test string is the same length as the table entry before
doing a strncasecmp to avoid doing just a prefix test.
---
 lib/utils.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/utils.c b/lib/utils.c
index 0ec7626..83d424f 100644
--- a/lib/utils.c
+++ b/lib/utils.c
@@ -993,7 +993,8 @@ int __str2flags(const char *buf, const struct trans_tbl *tbl, size_t tbl_len)
 		t = strchr(p, ',');
 		len = t ? t - p : strlen(p);
 		for (i = 0; i < tbl_len; i++)
-			if (!strncasecmp(tbl[i].a, p, len))
+			if (len == strlen(tbl[i].a) &&
+			    !strncasecmp(tbl[i].a, p, len))
 				flags |= tbl[i].i;
 
 		if (!t)
-- 
cgit v0.12