Macro argument should be enclosed in parentheses

twogood · Dec 2, 2017 · 917d3fe · 917d3fe
1 parent 5db827c
commit 917d3fe
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/internal.h b/lib/internal.h
@@ -95,12 +95,12 @@ uint8_t* unshield_header_get_buffer(Header* header, uint32_t offset);
    Macros for safer development
  */
 
-#define FREE(ptr)       { if (ptr) { free(ptr); ptr = NULL; } }
+#define FREE(ptr)       { if (ptr) { free(ptr); (ptr) = NULL; } }
 #define STRDUP(str)     ((str) ? strdup(str) : NULL)
 #define NEW(type, count)      ((type*)calloc(count, sizeof(type)))
 #define NEW1(type)      ((type*)calloc(1, sizeof(type)))
-#define FCLOSE(file)    if (file) { fclose(file); file = NULL; }
-#define FSIZE(file)     (file ? unshield_fsize(file) : 0)
+#define FCLOSE(file)    if (file) { fclose(file); (file) = NULL; }
+#define FSIZE(file)     ((file) ? unshield_fsize(file) : 0)
 #define STREQ(s1,s2)    (0 == strcmp(s1,s2))
 
 #if WORDS_BIGENDIAN