diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-11-07 03:08:44 -0500 |
---|---|---|
committer | Pawel Moll <pawel.moll@arm.com> | 2014-04-24 06:39:06 -0400 |
commit | 6b4ed8b00e93bd31f24a25f59ed8d1b808d0cc00 (patch) | |
tree | 2570de1eabab44273210e6d3e3bff4749f5840dd | |
parent | a798c10faf62a505d24e5f6213fbaf904a39623f (diff) |
clk: vexpress: NULL dereference on error path
If the allocation fails then we dereference the NULL in the error path.
Just return directly.
Fixes: ed27ff1db869 ('clk: Versatile Express clock generators ("osc") driver')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Pawel Moll <pawel.moll@arm.com>
-rw-r--r-- | drivers/clk/versatile/clk-vexpress-osc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/clk/versatile/clk-vexpress-osc.c b/drivers/clk/versatile/clk-vexpress-osc.c index 2dc8b41a339d..a535c7bf8574 100644 --- a/drivers/clk/versatile/clk-vexpress-osc.c +++ b/drivers/clk/versatile/clk-vexpress-osc.c | |||
@@ -102,7 +102,7 @@ void __init vexpress_osc_of_setup(struct device_node *node) | |||
102 | 102 | ||
103 | osc = kzalloc(sizeof(*osc), GFP_KERNEL); | 103 | osc = kzalloc(sizeof(*osc), GFP_KERNEL); |
104 | if (!osc) | 104 | if (!osc) |
105 | goto error; | 105 | return; |
106 | 106 | ||
107 | osc->func = vexpress_config_func_get_by_node(node); | 107 | osc->func = vexpress_config_func_get_by_node(node); |
108 | if (!osc->func) { | 108 | if (!osc->func) { |