fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-08 09:08:10 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

intel/tools: fix invalid type in argument to printf

Browse source 
$2 is exp2, exp2 is defined to be llint and llint is defined to be
unsigned long long int.

Fixes error reported by Coverity:
CID 1451141: Invalid type in argument to printf format specifier (PRINTF_ARGS)

Fixes: 70308a5a8a ("intel/tools: New i965 instruction assembler tool")

Signed-off-by: Marcin Ślusarz <marcin.slusarz@intel.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/7351>
(cherry picked from commit e96f33cd30)
This commit is contained in:
Marcin Ślusarz 2020-10-28 17:08:54 +01:00 • committed by Dylan Baker
parent b540aaa0a8
commit 3e2245c454
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
.pick_status.json
View file

@ -850,7 +850,7 @@
"description": "intel/tools: fix invalid type in argument to printf",
"nominated": true,
"nomination_type": 1,
"resolution": 0,
"resolution": 1,
"master_sha": null,
"because_sha": "70308a5a8a801a960bb9f45fb597f80b77c51014"
},

2
src/intel/tools/i965_gram.y
View file

@ -2185,7 +2185,7 @@ execsize:
| LPAREN exp2 RPAREN
{
if ($2 > 32 || !isPowerofTwo($2))
error(&@2, "Invalid execution size %d\n", $2);
error(&@2, "Invalid execution size %llu\n", $2);
$$ = cvt($2) - 1;
}