Commit e75f5634 authored by Dale Zhao's avatar Dale Zhao Committed by Alex Deucher

drm/amd/display: fix type mismatch error for return variable

It is possible for ret_vsnprintf to be assigned negative value in
error cases. As an unsigned variable, negative values which
are stored in their 2's complement form gets treated as a positive
number. This will led to treating bad cases as good ones.
eg: -1 gets stored as 0xFFFFFFFF on a 32 bit system
Signed-off-by: default avatarDale Zhao <dale.zhao@amd.com>
Reviewed-by: default avatarYongqiang Sun <yongqiang.sun@amd.com>
Acked-by: default avatarAnson Jacob <Anson.Jacob@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 91a51fbf
...@@ -47,7 +47,7 @@ ...@@ -47,7 +47,7 @@
unsigned int snprintf_count(char *pBuf, unsigned int bufSize, char *fmt, ...) unsigned int snprintf_count(char *pBuf, unsigned int bufSize, char *fmt, ...)
{ {
unsigned int ret_vsnprintf; int ret_vsnprintf;
unsigned int chars_printed; unsigned int chars_printed;
va_list args; va_list args;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment