On 10/6/21 16:34, Dan Carpenter wrote: > The last byte of "pad" is used without being initialized. > > Fixes: 55dba3120fbc ("libata: update ->data_xfer hook for ATAPI") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > drivers/ata/pata_legacy.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/ata/pata_legacy.c b/drivers/ata/pata_legacy.c > index c3e6592712c4..b53f0e09783d 100644 > --- a/drivers/ata/pata_legacy.c > +++ b/drivers/ata/pata_legacy.c > @@ -352,7 +352,7 @@ static unsigned int pdc_data_xfer_vlb(struct ata_queued_cmd *qc, > iowrite32_rep(ap->ioaddr.data_addr, buf, buflen >> 2); > > if (unlikely(slop)) { > - __le32 pad; > + __le32 pad = 0; > if (rw == READ) { > pad = cpu_to_le32(ioread32(ap->ioaddr.data_addr)); > memcpy(buf + buflen - slop, &pad, slop); > @@ -742,7 +742,7 @@ static unsigned int vlb32_data_xfer(struct ata_queued_cmd *qc, > ioread32_rep(ap->ioaddr.data_addr, buf, buflen >> 2); > > if (unlikely(slop)) { > - __le32 pad; > + __le32 pad = 0; > if (rw == WRITE) { > memcpy(&pad, buf + buflen - slop, slop); > iowrite32(le32_to_cpu(pad), ap->ioaddr.data_addr); > Applied to for-5.15-fixes. Thanks ! (Note: I added a blank line after the pad declaration to follow kernel style). -- Damien Le Moal Western Digital Research