diff options
author | Li Xiaoming <lixm.fnst@cn.fujitsu.com> | 2019-08-21 16:53:37 +0800 |
---|---|---|
committer | Jan-Simon Moeller <jsmoeller@linuxfoundation.org> | 2019-08-22 11:15:26 +0000 |
commit | 1eab9326969b3c56a233f4285bb716a3939a703f (patch) | |
tree | cbb24100541a5e93dadfbddc7d49a97661e339b9 /videoutils | |
parent | 94122a8f011fc629662931072aeb411f38f96e4a (diff) |
Remove potential overflow risk & improve indent
The patch below does:
1) As d_name type may hold 255(NAME_MAX) characters at most,
so I enlarge the array length to include the proper size.
maxX and maxY are s32(int32) type(-2,147,483,648 ~ +2,147,483,647),
which will cost 11 characters at most when writing to the arrary.
Also silence the warning.
2) Replace sprintf with snprintf, a safer version;
3) Improve indent;
4) Remove debug sentence.
Bug-AGL: SPEC-2422
Change-Id: I6d83a92a3138decd2bc9b750b9659d581a9f1b6b
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>
Diffstat (limited to 'videoutils')
0 files changed, 0 insertions, 0 deletions