Re: [PATCH 1/2] mmc: Remove redundant null check before kfree in sdio_bus.c
On 20 November 2012 14:43, Sachin Kamat sachin.ka...@linaro.org wrote: kfree on a null pointer is a no-op. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mmc/core/sdio_bus.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/core/sdio_bus.c b/drivers/mmc/core/sdio_bus.c index 6bf6879..fdcf9e3 100644 --- a/drivers/mmc/core/sdio_bus.c +++ b/drivers/mmc/core/sdio_bus.c @@ -258,8 +258,7 @@ static void sdio_release_func(struct device *dev) sdio_free_func_cis(func); - if (func-info) - kfree(func-info); + kfree(func-info); kfree(func); } -- 1.7.4.1 ping... -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] mmc: Remove redundant null check before kfree in sdio_bus.c
Hi Sachin, On Tue, Nov 20 2012, Sachin Kamat wrote: kfree on a null pointer is a no-op. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mmc/core/sdio_bus.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/core/sdio_bus.c b/drivers/mmc/core/sdio_bus.c index 6bf6879..fdcf9e3 100644 --- a/drivers/mmc/core/sdio_bus.c +++ b/drivers/mmc/core/sdio_bus.c @@ -258,8 +258,7 @@ static void sdio_release_func(struct device *dev) sdio_free_func_cis(func); - if (func-info) - kfree(func-info); + kfree(func-info); kfree(func); } Thanks, pushed both patches to mmc-next for 3.8. - Chris. -- Chris Ball c...@laptop.org http://printf.net/ One Laptop Per Child -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] mmc: Remove redundant null check before kfree in sdio_bus.c
Hi Chris Any comments on these 2 patches? On 20 November 2012 14:43, Sachin Kamat sachin.ka...@linaro.org wrote: kfree on a null pointer is a no-op. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mmc/core/sdio_bus.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/core/sdio_bus.c b/drivers/mmc/core/sdio_bus.c index 6bf6879..fdcf9e3 100644 --- a/drivers/mmc/core/sdio_bus.c +++ b/drivers/mmc/core/sdio_bus.c @@ -258,8 +258,7 @@ static void sdio_release_func(struct device *dev) sdio_free_func_cis(func); - if (func-info) - kfree(func-info); + kfree(func-info); kfree(func); } -- 1.7.4.1 -- With warm regards, Sachin -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH 1/2] mmc: Remove redundant null check before kfree in sdio_bus.c
kfree on a null pointer is a no-op. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/mmc/core/sdio_bus.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/core/sdio_bus.c b/drivers/mmc/core/sdio_bus.c index 6bf6879..fdcf9e3 100644 --- a/drivers/mmc/core/sdio_bus.c +++ b/drivers/mmc/core/sdio_bus.c @@ -258,8 +258,7 @@ static void sdio_release_func(struct device *dev) sdio_free_func_cis(func); - if (func-info) - kfree(func-info); + kfree(func-info); kfree(func); } -- 1.7.4.1 -- To unsubscribe from this list: send the line unsubscribe linux-mmc in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html