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