diff options
| author | Haojian Zhuang <haojian.zhuang@linaro.org> | 2013-10-20 15:42:02 +0800 |
|---|---|---|
| committer | Haojian Zhuang <haojian.zhuang@linaro.org> | 2013-11-15 10:22:57 +0800 |
| commit | 75a49a91585ef7b2afc2c193c0c534285a9ef49c (patch) | |
| tree | e945121c71d76117864dd0667f3a2aba4d23452c | |
| parent | 77ef84c4b8603784a41e6db7fbd65742285e0960 (diff) | |
clk: mux: fix the return value of get_parent
If there're no parents on the clock mux node, it should return 0, not an
errnor code.
The orphan clock node is using get_parent() to get the index of its
parents in__clk_init(). If the return value is negative, it'll be
transformed to a u8 value (clk_mux_get_parent()). Then the orphan clock
node tries to get its parent with the invalid index. It will only cause
accessing memory beyond border.
Signed-off-by: Haojian Zhuang <haojian.zhuang@linaro.org>
| -rw-r--r-- | drivers/clk/clk-mux.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/clk/clk-mux.c b/drivers/clk/clk-mux.c index 4f96ff3ba728..cc06015aee0a 100644 --- a/drivers/clk/clk-mux.c +++ b/drivers/clk/clk-mux.c @@ -51,7 +51,7 @@ static u8 clk_mux_get_parent(struct clk_hw *hw) for (i = 0; i < num_parents; i++) if (mux->table[i] == val) return i; - return -EINVAL; + return 0; } if (val && (mux->flags & CLK_MUX_INDEX_BIT)) @@ -61,7 +61,7 @@ static u8 clk_mux_get_parent(struct clk_hw *hw) val--; if (val >= num_parents) - return -EINVAL; + return 0; return val; } |