Re: [PATCH v4 07/17] fs/kernel_read_file: Switch buffer size arg to size_t

On Wed, 29 Jul 2020, Kees Cook wrote:

In preparation for further refactoring of kernel_read_file*(), rename the "max_size" argument to the more accurate "buf_size", and correct its type to size_t. Add kerndoc to explain the specifics of how the arguments will be used. Note that with buf_size now size_t, it can no longer be negative (and was never called with a negative value). Adjust callers to use it as a "maximum size" when *buf is NULL.

Acked-by: Scott Branden scott.branden@broadcom.com Reviewed-by: Mimi Zohar zohar@linux.ibm.com Reviewed-by: Luis Chamberlain mcgrof@kernel.org Signed-off-by: Kees Cook keescook@chromium.org

Reviewed-by: James Morris jamorris@linux.microsoft.com