Add wrapper for os_free in order to solve the problem of the optional…

… semicolon
ruuvi · Dec 8, 2021 · 57c75ea · 57c75ea
1 parent 0cb87a3
commit 57c75ea
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/include/os_malloc.h b/include/os_malloc.h
@@ -80,11 +80,14 @@ os_free_internal(void *ptr);
  * @brief os_free - is a wrap for 'free' which automatically sets pointer to NULL after the memory freeing.
  */
 #define os_free(ptr) \
-    if (NULL != (ptr)) \
+    do \
     { \
-        os_free_internal((void *)(ptr)); \
-        ptr = NULL; \
-    }
+        if (NULL != (ptr)) \
+        { \
+            os_free_internal((void *)(ptr)); \
+            ptr = NULL; \
+        } \
+    } while (0)
 
 #ifdef __cplusplus
 }