Be a little smarter about allowing VALUE names that start

	with digits.  It turns out there's a bunch of these in the
	dictionaries that haven't yet been caught, so being smarter
	here helps a lot.

diff --git a/src/lib/valuepair.c b/src/lib/valuepair.c
index c8cfe25..acb8c6a 100644 (file)
--- a/src/lib/valuepair.c
+++ b/src/lib/valuepair.c
@@ -773,15 +773,14 @@ VALUE_PAIR *pairparsevalue(VALUE_PAIR *vp, const char *value)
                        break;
                case PW_TYPE_INTEGER:
                        /*
-                        *      If it starts with a digit, it must
-                        *      be a number (or a range).
-                        *
                         *      Note that ALL integers are unsigned!
                         */
-                       if (isdigit((int) *value)) {
+                       if (isdigit((int) value[0]) &&
+                           (strspn(value, "0123456789") == strlen(value))) {
                                vp->lvalue = (uint32_t) strtoul(value, NULL, 10);
                                vp->length = 4;
                        }
+
                        /*
                         *      Look for the named value for the given
                         *      attribute.