On 8/25/20 3:28 PM, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > > Don't leak kernel memory contents into the shortform attr fork. > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> I noticed this too, thanks. thought I wonder if a) others lurk and b) if we should just be memsetting if_data to zero to avoid the need to carefully initialize all of everything always? Anyway it fixes the problem we noticed so Reviewed-by: Eric Sandeen <sandeen@xxxxxxxxxx> > --- > fs/xfs/libxfs/xfs_attr_leaf.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/fs/xfs/libxfs/xfs_attr_leaf.c b/fs/xfs/libxfs/xfs_attr_leaf.c > index 8623c815164a..e1a3d225a77d 100644 > --- a/fs/xfs/libxfs/xfs_attr_leaf.c > +++ b/fs/xfs/libxfs/xfs_attr_leaf.c > @@ -656,6 +656,7 @@ xfs_attr_shortform_create( > hdr = (xfs_attr_sf_hdr_t *)ifp->if_u1.if_data; > hdr->count = 0; > hdr->totsize = cpu_to_be16(sizeof(*hdr)); > + hdr->padding = 0; > xfs_trans_log_inode(args->trans, dp, XFS_ILOG_CORE | XFS_ILOG_ADATA); > } > >
