Message ID | 20200714110027.301728-1-christophe.jaillet@wanadoo.fr |
---|---|
State | Accepted |
Delegated to: | David Miller |
Headers | show |
Series | hippi: Fix a size used in a 'pci_free_consistent()' in an error handling path | expand |
From: Christophe JAILLET <christophe.jaillet@wanadoo.fr> Date: Tue, 14 Jul 2020 13:00:27 +0200 > The size used when calling 'pci_alloc_consistent()' and > 'pci_free_consistent()' should match. > > Fix it and have it consistent with the corresponding call in 'rr_close()'. > > Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") > Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> Applied, thank you.
diff --git a/drivers/net/hippi/rrunner.c b/drivers/net/hippi/rrunner.c index 2a6ec5394966..d0d5100eeb0c 100644 --- a/drivers/net/hippi/rrunner.c +++ b/drivers/net/hippi/rrunner.c @@ -1242,7 +1242,7 @@ static int rr_open(struct net_device *dev) rrpriv->info = NULL; } if (rrpriv->rx_ctrl) { - pci_free_consistent(pdev, sizeof(struct ring_ctrl), + pci_free_consistent(pdev, 256 * sizeof(struct ring_ctrl), rrpriv->rx_ctrl, rrpriv->rx_ctrl_dma); rrpriv->rx_ctrl = NULL; }
The size used when calling 'pci_alloc_consistent()' and 'pci_free_consistent()' should match. Fix it and have it consistent with the corresponding call in 'rr_close()'. Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2") Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> --- drivers/net/hippi/rrunner.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)