lib: Clarify the return value of strnlen_user()
authorJan Kara <jack@suse.cz>
Wed, 3 Jun 2015 13:50:35 +0000 (15:50 +0200)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 3 Jun 2015 15:00:22 +0000 (08:00 -0700)
strnlen_user() can return a number in a range 0 to count +
sizeof(unsigned long) - 1. Clarify the comment at the top of the
function so that users don't think the function returns at most count+1.

Signed-off-by: Jan Kara <jack@suse.cz>
[ Also added commentary about preferably not using this function ]
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
lib/strnlen_user.c

index 11649615c50504cb5976d58c8bf012d563bf2bbc..fe9a32591c2498b6266c2fc4753521fb469c876a 100644 (file)
@@ -90,8 +90,15 @@ static inline long do_strnlen_user(const char __user *src, unsigned long count,
  * Get the size of a NUL-terminated string in user space.
  *
  * Returns the size of the string INCLUDING the terminating NUL.
- * If the string is too long, returns 'count+1'.
+ * If the string is too long, returns a number larger than @count. User
+ * has to check the return value against "> count".
  * On exception (or invalid count), returns 0.
+ *
+ * NOTE! You should basically never use this function. There is
+ * almost never any valid case for using the length of a user space
+ * string, since the string can be changed at any time by other
+ * threads. Use "strncpy_from_user()" instead to get a stable copy
+ * of the string.
  */
 long strnlen_user(const char __user *str, long count)
 {