> > Checking ./drivers/mmc/card/queue.c... > > [./drivers/mmc/card/queue.c:212]: (style) Redundant condition. It is safe > > to deallocate a NULL pointer > > [./drivers/mmc/card/queue.c:215]: (style) Redundant condition. It is safe > > to deallocate a NULL pointer > > [./drivers/mmc/card/queue.c:208]: (style) Redundant condition. It is safe > > to deallocate a NULL pointer > > [./drivers/mmc/card/queue.c:239]: (style) Redundant condition. It is safe > > to deallocate a NULL pointer > > [./drivers/mmc/card/queue.c:246]: (style) Redundant condition. It is safe > > to deallocate a NULL pointer > > This report appears to be using a kernel config with > CONFIG_MMC_BLOCK_BOUNCE=y. > What does cppcheck report for the CONFIG_MMC_BLOCK_BOUNCE=n case? I believe > you will find that it is not sage to remove those conditions for the > CONFIG_MMC_BLOCK_BOUNCE=n case. >
How is that? kfree(NULL) is legal. -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ |
signature.asc
Description: Digital signature
