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:16:16 +0000 |
commit | 81006a03b9014ec482200a7fedb189b97abbff4a (patch) | |
tree | a3faa8c3bedbc4ed2a7065cbc13a389549734b89 /videoutils | |
parent | 8310076d9255f0a54d6b9c0064b3e08469970497 (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