On Tue, Oct 20, 2020 at 06:47:06PM +0200, laniel_fran...@privacyrequired.com 
wrote:
> From: Francis Laniel <laniel_fran...@privacyrequired.com>
> 
> nla_strlcpy now returns -E2BIG if src was truncated when written to dst.
> It also returns this error value if dstsize is 0 or higher than INT_MAX.
> 
> For example, if src is "foo\0" and dst is 3 bytes long, the result will be:
> 1. "foG" after memcpy (G means garbage).
> 2. "fo\0" after memset.
> 3. -E2BIG is returned because src was not completely written into dst.
> 
> The callers of nla_strlcpy were modified to take into account this 
> modification.
> 
> Signed-off-by: Francis Laniel <laniel_fran...@privacyrequired.com>

This looks correct to me. Thanks for the respin!

Reviewed-by: Kees Cook <keesc...@chromium.org>

-- 
Kees Cook

Reply via email to