nhyatt/putty-source
1
0
Fork 0
mirror of https://git.tartarus.org/simon/putty.git synced 2025-07-02 20:12:48 -05:00
Code Issues Projects Releases Wiki Activity

windows/unicode.c: tighten up a bounds check.

Browse Source 
Coverity points out that if we refer to cp_list[codepage - 65536], we
ought to have ensured that codepage - 65536 was _less_ than
lenof(cp_list), not just less or equal.
This commit is contained in:
Simon Tatham
2022-09-07 14:18:21 +01:00
parent 1f6d93f0c8
commit 3442fb1aeb
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
windows/unicode.c
View File

@ -531,7 +531,7 @@ static reverse_mapping *get_reverse_mapping(int codepage)
if (codepage < 65536) if (codepage < 65536)
return NULL; return NULL;
if (codepage > 65536 + lenof(cp_list)) if (codepage >= 65536 + lenof(cp_list))
return NULL; return NULL;
const struct cp_list_item *cp = &cp_list[codepage - 65536]; const struct cp_list_item *cp = &cp_list[codepage - 65536];
if (!cp->cp_table) if (!cp->cp_table)