From 718da3ffffd1181a0b5b2d590214bc408ca44117 Mon Sep 17 00:00:00 2001
From: Vincent Sanders <vince@kyllikki.org>
Date: Sun, 5 Mar 2017 12:11:48 +0000
Subject: make urldb internal parse macros less prone to control flow errors

fixes issue highlighted by coverity (CID 1361696)
---
 content/urldb.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

(limited to 'content')

diff --git a/content/urldb.c b/content/urldb.c
index 60944f7e5..c1c39c145 100644
--- a/content/urldb.c
+++ b/content/urldb.c
@@ -3796,7 +3796,7 @@ void urldb_load_cookies(const char *filename)
 	if (!fp)
 		return;
 
-#define FIND_T {							\
+#define FIND_T do {							\
 		for (; *p && *p != '\t'; p++)				\
 			; /* do nothing */				\
 		if (p >= end) {						\
@@ -3804,16 +3804,16 @@ void urldb_load_cookies(const char *filename)
 			continue;					\
 		}							\
 		*p++ = '\0';						\
-}
+	} while(0)
 
-#define SKIP_T {							\
+#define SKIP_T do {							\
 		for (; *p && *p == '\t'; p++)				\
 			; /* do nothing */				\
 		if (p >= end) {						\
 			LOG("Overran input");				\
 			continue;					\
 		}							\
-}
+	} while(0)
 
 	while (fgets(s, sizeof s, fp)) {
 		char *p = s, *end = 0,
-- 
cgit v1.2.3