]> git.proxmox.com Git - pve-firewall.git/commitdiff
make LEPRINT* macros safe to use with if/else pairs
authorWolfgang Bumiller <w.bumiller@proxmox.com>
Tue, 26 Jan 2016 09:22:51 +0000 (10:22 +0100)
committerDietmar Maurer <dietmar@proxmox.com>
Tue, 26 Jan 2016 10:03:15 +0000 (11:03 +0100)
src/pvefw-logger.c

index b7a2fb2983258e8cfc14d322151e0913e642e6bf..19beb18406b305d8067e1a9f7d146179178e8b5e 100644 (file)
@@ -175,8 +175,17 @@ queue_log_entry(struct log_entry *le)
 }
 
 
-#define LEPRINTF(format, ...) { if (le->len < LE_MAX) le->len += snprintf(le->buf + le->len, LE_MAX - le->len, format, ##__VA_ARGS__); }
-#define LEPRINTTIME(sec) { time_t tmp_sec = sec; if (le->len < (LE_MAX - 30)) le->len += strftime(le->buf + le->len, LE_MAX - le->len, "%d/%b/%Y:%H:%M:%S %z ", localtime(&tmp_sec)); }
+#define LEPRINTF(format, ...) \
+    do { \
+        if (le->len < LE_MAX) \
+            le->len += snprintf(le->buf + le->len, LE_MAX - le->len, format, ##__VA_ARGS__); \
+    } while (0)
+#define LEPRINTTIME(sec) \
+    do { \
+        time_t tmp_sec = sec; \
+        if (le->len < (LE_MAX - 30)) \
+            le->len += strftime(le->buf + le->len, LE_MAX - le->len, "%d/%b/%Y:%H:%M:%S %z ", localtime(&tmp_sec)); \
+    } while (0)
 
 static void
 log_status_message(guint loglevel, const char *fmt, ...)