jsoninfo: helpers: use sizeof() for uuid size
authorFerry Huberts <ferry.huberts@pelagic.nl>
Wed, 2 Dec 2015 15:49:28 +0000 (16:49 +0100)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 3 Dec 2015 11:00:09 +0000 (12:00 +0100)
And make its size a bit less strange.
Plus an extra minor simplification.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
lib/jsoninfo/src/olsrd_jsoninfo_helpers.c
lib/jsoninfo/src/olsrd_jsoninfo_helpers.h

index e61c0c1..d8526a0 100644 (file)
@@ -50,7 +50,7 @@
 #include <fcntl.h>
 #endif /* __linux__ */
 
-char uuid[UUIDLEN + 1];
+char uuid[UUIDLEN];
 
 /* JSON support functions */
 
@@ -196,11 +196,11 @@ int read_uuid_from_file(const char * name, const char *file) {
 
   f = fopen(file, "r");
   olsr_printf(1, "(%s) Reading UUID from '%s'\n", name, file);
-  if (f == NULL) {
+  if (!f) {
     olsr_printf(1, "(%s) Could not open '%s': %s\n", name, file, strerror(errno));
     return -1;
   }
-  chars = fread(uuid, 1, UUIDLEN, f);
+  chars = fread(uuid, 1, sizeof(uuid) - 1, f);
   if (chars > 0) {
     uuid[chars] = '\0'; /* null-terminate the string */
 
index 1d154da..633cbcf 100644 (file)
@@ -48,7 +48,7 @@
 #include "common/autobuf.h"
 
 #define UUIDLEN 256
-extern char uuid[UUIDLEN + 1];
+extern char uuid[UUIDLEN];
 
 void abuf_json_reset_entry_number_and_depth(void);