INN commit: trunk/storage/cnfs (cnfs.c)

[INN Commit]

    Date: Saturday, May 23, 2015 @ 04:35:34
  Author: iulius
Revision: 9873

storage/cnfs/cnfs.c:  don't read uninitialised cycbuffname

Thanks to Richard Kettlewell for the patch.

Modified:
  trunk/storage/cnfs/cnfs.c

--------+
 cnfs.c |   10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

Modified: cnfs.c
===================================================================
--- cnfs.c	2015-05-23 11:30:49 UTC (rev 9872)
+++ cnfs.c	2015-05-23 11:35:34 UTC (rev 9873)
@@ -128,9 +128,13 @@
     uint32_t	uint32;
 
     if (cycbuffname == NULL || blk == NULL || cycnum == NULL) {
-        warn("CNFS: BreakToken: invalid argument: %s", cycbuffname);
-	SMseterror(SMERR_INTERNAL, "BreakToken: invalid argument");
-	return false;
+        if (cycbuffname == NULL) {
+            warn("CNFS: BreakToken: invalid argument");
+        } else {
+            warn("CNFS: BreakToken: invalid argument: %s", cycbuffname);
+        }
+        SMseterror(SMERR_INTERNAL, "BreakToken: invalid argument");
+        return false;
     }
     memcpy(cycbuffname, token.token, CNFSMAXCYCBUFFNAME);
     *(cycbuffname + CNFSMAXCYCBUFFNAME) = '\0';	/* Just to be paranoid */