On 21/11/2017 18:49, Ville Syrjala wrote:
From: Ville Syrjälä <[email protected]>Bump the snprintf() buffer size to 32 bytes to silence gcc. intel_watermark.c:203:57: warning: ‘%c’ directive output may be truncated writing 1 byte into a region of size between 0 and 9 [-Wformat-truncation=] snprintf(reg_name, sizeof(reg_name), "PLANE_WM_%1d_%c_%1d", ^~ intel_watermark.c:203:43: note: directive argument in the range [0, 8] snprintf(reg_name, sizeof(reg_name), "PLANE_WM_%1d_%c_%1d", ^~~~~~~~~~~~~~~~~~~~~ etc. Cc: Dhinakaran Pandiyan <[email protected]> Cc: Tvrtko Ursulin <[email protected]> Reported-by: Tvrtko Ursulin <[email protected]> Signed-off-by: Ville Syrjälä <[email protected]> --- tools/intel_watermark.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/intel_watermark.c b/tools/intel_watermark.c index c09772eea334..1d087b49f8c2 100644 --- a/tools/intel_watermark.c +++ b/tools/intel_watermark.c @@ -163,7 +163,7 @@ static void skl_wm_dump(void) uint32_t wm_trans[num_pipes][max_planes]; uint32_t buf_cfg[num_pipes][max_planes]; uint32_t wm_linetime[num_pipes]; - char reg_name[20]; + char reg_name[32];intel_register_access_init(intel_get_pci_device(), 0, -1);
Reviewed-by: Tvrtko Ursulin <[email protected]> Tested-by: Tvrtko Ursulin <[email protected]> Regards, Tvrtko _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
