On May 29 2016 or thereabouts, Corey Minyard wrote: > From: Corey Minyard <cminy...@mvista.com> > > Don't leave a pointer to some external buffer lying around. > > Signed-off-by: Corey Minyard <cminy...@mvista.com> > ---
Looks good to me. Reviewed-by: Benjamin Tissoires <benjamin.tissoi...@redhat.com> > drivers/i2c/busses/i2c-i801.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c > index 70da60a..bb15356 100644 > --- a/drivers/i2c/busses/i2c-i801.c > +++ b/drivers/i2c/busses/i2c-i801.c > @@ -598,6 +598,7 @@ static int i801_block_transaction_byte_by_byte(struct > i801_priv *priv, > dev_warn(&priv->pci_dev->dev, > "Timeout waiting for interrupt!\n"); > } > + priv->data = NULL; > priv->status = 0; > return status; > }
