From 505c80376538994ee63aa2e4a2b5021e107ee250 Mon Sep 17 00:00:00 2001 From: Vincent Penquerc'h Date: Thu, 10 Apr 2014 11:19:26 +0100 Subject: [PATCH] sdpmessage: fix multi statement macros Wasn't playing nice with an if statement below. Coverity 1139767 --- gst-libs/gst/sdp/gstsdpmessage.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/gst-libs/gst/sdp/gstsdpmessage.c b/gst-libs/gst/sdp/gstsdpmessage.c index 844819966e..9e2565c5c7 100644 --- a/gst-libs/gst/sdp/gstsdpmessage.c +++ b/gst-libs/gst/sdp/gstsdpmessage.c @@ -2796,8 +2796,10 @@ gst_sdp_parse_line (SDPContext * c, gchar type, gchar * buffer) gchar str[8192]; gchar *p = buffer; -#define READ_STRING(field) read_string (str, sizeof (str), &p); REPLACE_STRING (field, str) -#define READ_UINT(field) read_string (str, sizeof (str), &p); field = strtoul (str, NULL, 10) +#define READ_STRING(field) \ + do { read_string (str, sizeof (str), &p); REPLACE_STRING (field, str); } while (0) +#define READ_UINT(field) \ + do { read_string (str, sizeof (str), &p); field = strtoul (str, NULL, 10); } while (0) switch (type) { case 'v':