Changeset 881 in openpam

Timestamp:

Jan 9, 2016, 6:29:32 PM (5 years ago)

Author:

Dag-Erling Smørgrav

Message:

Fix off-by-one bug: we forgot to account for the terminating NUL when
checking the length of the label.

File:

• trunk/lib/liboath/oath_key_create.c (modified) (1 diff)

trunk/lib/liboath/oath_key_create.c

@@ -56 +56 @@
         int labellen;
 
-        /* check label */
+        /* check label length */
         if (label == NULL ||
-            (labellen = strlen(label)) > OATH_MAX_LABELLEN)
+            (labellen = strlen(label)) >= OATH_MAX_LABELLEN)
                 return (NULL);