Messages in this thread Patch in this message | | | From | Tomeu Vizoso <> | Subject | [PATCH v3 3/8] clk: Don't try to use a struct clk* after it could have been freed | Date | Thu, 9 Oct 2014 17:01:11 +0200 |
| |
As __clk_release could call kfree on clk and then we wouldn't have a safe way of getting the module that owns the clock.
Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com> --- drivers/clk/clk.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index d0712b7..40aa7ad 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -2268,14 +2268,16 @@ int __clk_get(struct clk *clk) void __clk_put(struct clk *clk) { + struct module *owner; + if (!clk || WARN_ON_ONCE(IS_ERR(clk))) return; clk_prepare_lock(); + owner = clk->owner; kref_put(&clk->ref, __clk_release); + module_put(owner); clk_prepare_unlock(); - - module_put(clk->owner); } /*** clk rate change notifiers ***/ -- 1.9.3
| |