diff options
author | Lucas Stach <dev@lynxeye.de> | 2014-02-07 09:48:43 +0100 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2014-02-10 08:49:55 +0100 |
commit | aa6694b5422dc2d598ce09179b9ad9de8fa403b4 (patch) | |
tree | f16d0789b94ae5cf0efc6d087d41237dc10b4dd8 /lib | |
parent | 34ab8ddd04ae08e8bca6d277b4f692f4f303c20d (diff) | |
download | barebox-aa6694b5422dc2d598ce09179b9ad9de8fa403b4.tar.gz barebox-aa6694b5422dc2d598ce09179b9ad9de8fa403b4.tar.xz |
remove redundant NULL check on free
free() already checks the pointer to be non
NULL. No need to do it again.
Signed-off-by: Lucas Stach <dev@lynxeye.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/parameter.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/lib/parameter.c b/lib/parameter.c index c5c6426bb2..a0bae3e90a 100644 --- a/lib/parameter.c +++ b/lib/parameter.c @@ -108,8 +108,7 @@ int dev_set_param(struct device_d *dev, const char *name, const char *val) int dev_param_set_generic(struct device_d *dev, struct param_d *p, const char *val) { - if (p->value) - free(p->value); + free(p->value); if (!val) { p->value = NULL; return 0; |