On 22/05/25 11:01AM, Shuqi Zhang wrote: > Replace kmalloc + memcpy with kmemdup() Thanks for the patch. A straight forward conversion. Looks good to me. Feel free to add - Reviewed-by: Ritesh Harjani <ritesh.list@xxxxxxxxx> > > Signed-off-by: Shuqi Zhang <zhangshuqi3@xxxxxxxxxx> > --- > fs/ext4/xattr.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/fs/ext4/xattr.c b/fs/ext4/xattr.c > index 042325349098..564e28a1aa94 100644 > --- a/fs/ext4/xattr.c > +++ b/fs/ext4/xattr.c > @@ -1895,11 +1895,10 @@ ext4_xattr_block_set(handle_t *handle, struct inode *inode, > > unlock_buffer(bs->bh); > ea_bdebug(bs->bh, "cloning"); > - s->base = kmalloc(bs->bh->b_size, GFP_NOFS); > + s->base = kmemdup(BHDR(bs->bh), bs->bh->b_size, GFP_NOFS); > error = -ENOMEM; > if (s->base == NULL) > goto cleanup; > - memcpy(s->base, BHDR(bs->bh), bs->bh->b_size); > s->first = ENTRY(header(s->base)+1); > header(s->base)->h_refcount = cpu_to_le32(1); > s->here = ENTRY(s->base + offset); > -- > 2.31.1 >
