Changeset 52 in openpam

Timestamp:

Feb 13, 2002, 12:41:06 AM (18 years ago)

Author:

Dag-Erling Smørgrav

Message:

Oops, previous commit would cause sprintf() to overrun the malloc()ed
buffer by exactly one character. Add some slack.

Sponsored by: DARPA, NAI Labs

File:

• trunk/lib/openpam_log.c (modified) (1 diff)

trunk/lib/openpam_log.c

@@ -75 +75 @@
         for (len = strlen(fmt); len > 0 && isspace(fmt[len]); len--)
                 /* nothing */;
-        if ((format = malloc(strlen(func) + len + 8)) != NULL) {
+        if ((format = malloc(strlen(func) + len + 16)) != NULL) {
                 sprintf(format, "in %s(): %.*s\n", func, len, fmt);
                 vsyslog(priority, format, ap);