Jim Meyering <j...@meyering.net> writes: > From: Jim Meyering <meyer...@redhat.com> > > In both mp_pacl_listxattr and mp_dacl_listxattr, the uses of strncpy > were unnecessary, since at each point of use we know that the > NUL-terminated source bytes fit in the destination buffer. > Use memcpy in place of strncpy. > > Signed-off-by: Jim Meyering <meyer...@redhat.com>
Acked-by: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com> > --- > hw/9pfs/virtio-9p-posix-acl.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/hw/9pfs/virtio-9p-posix-acl.c b/hw/9pfs/virtio-9p-posix-acl.c > index a1948e3..c064017 100644 > --- a/hw/9pfs/virtio-9p-posix-acl.c > +++ b/hw/9pfs/virtio-9p-posix-acl.c > @@ -44,7 +44,8 @@ static ssize_t mp_pacl_listxattr(FsContext *ctx, const char > *path, > return -1; > } > > - strncpy(value, ACL_ACCESS, len); > + /* len includes the trailing NUL */ > + memcpy(value, ACL_ACCESS, len); > return 0; > } > > @@ -95,7 +96,8 @@ static ssize_t mp_dacl_listxattr(FsContext *ctx, const char > *path, > return -1; > } > > - strncpy(value, ACL_DEFAULT, len); > + /* len includes the trailing NUL */ > + memcpy(value, ACL_ACCESS, len); > return 0; > } > > -- > 1.7.10.1.487.ga3935e6