diff options
author | Kees Cook <keescook@chromium.org> | 2019-09-25 19:46:20 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2019-09-25 20:51:39 -0400 |
commit | 9a156466147b61504f4cbe97ea503e67c21e117a (patch) | |
tree | 25f41cd13e244c58bdf536c601225a05daa4caa9 /lib | |
parent | d1a445d3b86c9341ce7a0954c23be0edb5c9bec5 (diff) |
strscpy: reject buffer sizes larger than INT_MAX
As already done for snprintf(), add a check in strscpy() for giant (i.e.
likely negative and/or miscalculated) copy sizes, WARN, and error out.
Link: http://lkml.kernel.org/r/201907260928.23DE35406@keescook
Signed-off-by: Kees Cook <keescook@chromium.org>
Cc: Joe Perches <joe@perches.com>
Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Cc: Yann Droneaud <ydroneaud@opteya.com>
Cc: David Laight <David.Laight@aculab.com>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Stephen Kitt <steve@sk2.org>
Cc: Jann Horn <jannh@google.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/string.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/string.c b/lib/string.c index f7bc10da4259..cd7a10c19210 100644 --- a/lib/string.c +++ b/lib/string.c | |||
@@ -183,7 +183,7 @@ ssize_t strscpy(char *dest, const char *src, size_t count) | |||
183 | size_t max = count; | 183 | size_t max = count; |
184 | long res = 0; | 184 | long res = 0; |
185 | 185 | ||
186 | if (count == 0) | 186 | if (count == 0 || WARN_ON_ONCE(count > INT_MAX)) |
187 | return -E2BIG; | 187 | return -E2BIG; |
188 | 188 | ||
189 | #ifdef CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS | 189 | #ifdef CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS |