diff options
author | Julia Lawall <Julia.Lawall@lip6.fr> | 2015-04-04 16:59:29 +0200 |
---|---|---|
committer | Stephen Boyd <sboyd@codeaurora.org> | 2015-04-09 08:19:31 -0700 |
commit | e52786ac3ca721636a6fb0ae88521f5b9ece88a3 (patch) | |
tree | 4d38160ab42c14fa65ca356ada2b792447b0cc3b /drivers/clk/pxa/clk-pxa3xx.c | |
parent | 2f272e7b015c6d0f308d1a03e71557c962b10ce9 (diff) |
clk: versatile: test returned value
Put NULL test on the result of the previous call instead on one of its
arguments. A simplified version of the semantic match that finds this
problem is as follows (http://coccinelle.lip6.fr/):
// <smpl>
r@
expression *e1;
expression *e2;
identifier f;
statement S1,S2;
@@
e1 = f(...,e2,...);
(
if (e1 == NULL || ...) S1 else S2
|
*if (e2 == NULL || ...) S1 else S2
)
// </smpl>
Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Diffstat (limited to 'drivers/clk/pxa/clk-pxa3xx.c')
0 files changed, 0 insertions, 0 deletions