From: Joe Stringer Date: Fri, 12 Aug 2016 00:54:08 +0000 (-0700) Subject: ofproto-dpif-xlate: Fix VLOG_ERR_RL() call. X-Git-Tag: v2.12.3~4802 X-Git-Url: https://git.proxmox.com/?a=commitdiff_plain;h=26f858a9174dc4c2283e5d5f0d798d9a0c478e55;p=mirror_ovs.git ofproto-dpif-xlate: Fix VLOG_ERR_RL() call. a716ef9a7a73 ("ofproto-dpif-xlate: Log flow in XLATE_REPORT_ERROR.") inadvertantly broke build on clang due to improper passing of the ds cstring into the VLOG() function: error: format string is not a string literal (potentially insecure) [-Werror,-Wformat-security] XLATE_REPORT_ERROR(ctx, "over max translation depth %d", MAX_DEPTH); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ note: expanded from macro 'XLATE_REPORT_ERROR' VLOG_ERR_RL(&error_report_rl, ds_cstr(&ds)); \ Reported-by: Daniele Di Proietto Signed-off-by: Joe Stringer Acked-by: Daniele Di Proietto --- diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index 1e375b58f..f0a25011f 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofproto-dpif-xlate.c @@ -637,7 +637,7 @@ static struct vlog_rate_limit error_report_rl = VLOG_RATE_LIMIT_INIT(1, 5); ds_put_format(&ds, __VA_ARGS__); \ ds_put_cstr(&ds, ": "); \ flow_format(&ds, &ctx->base_flow); \ - VLOG_ERR_RL(&error_report_rl, ds_cstr(&ds)); \ + VLOG_ERR_RL(&error_report_rl, "%s", ds_cstr(&ds)); \ ds_destroy(&ds); \ } \ } while (0)