drivers/net/can: Correct NULL test
Test the just-allocated value for NULL rather than some other value. The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression x,y; statement S; @@ x = \(kmalloc\|kcalloc\|kzalloc\)(...); ( if ((x) == NULL) S | if ( - y + x == NULL) S ) // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Acked-by: Oliver Hartkopp <oliver@hartkopp.net> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
073bd90f03
commit
ce739b473c
|
@ -990,7 +990,7 @@ static int __devinit mcp251x_can_probe(struct spi_device *spi)
|
|||
goto error_tx_buf;
|
||||
}
|
||||
priv->spi_rx_buf = kmalloc(SPI_TRANSFER_BUF_LEN, GFP_KERNEL);
|
||||
if (!priv->spi_tx_buf) {
|
||||
if (!priv->spi_rx_buf) {
|
||||
ret = -ENOMEM;
|
||||
goto error_rx_buf;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue