drivers/staging/cptm1217/clearpad_tm1217.c: Correct call to input_free_device
commit311fda8e8b495e026fa1d24d9d5a211fc8be2aef
authorJulia Lawall <julia@diku.dk>
Wed, 18 May 2011 19:20:43 +0000 (18 21:20 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 18 May 2011 21:30:49 +0000 (18 14:30 -0700)
tree23e71e47ee0323f65ea586c432a7dc10ea894ef7
parent8041f92a204c06aa5bef5f42a5108e3886c0b215
drivers/staging/cptm1217/clearpad_tm1217.c: Correct call to input_free_device

This code is in a loop that currently is only executed once.  Because of
this property, the first block of code is currently actually correct.
Nevertheless, the comments associated with the code suggest that the loop
is planned to take more than one iteration in the future, and thus this
patch is made with that case in mind.

In the first block of code, there is currently an immediate abort from the
function.  It is changed to jump to the error handling code at fail, to be
able to unregister and free the resources allocated on previous iterations.

In the second block of code, the input_dev for the current iteration has
been allocated, but has not been registered.  It has also not been stored
in ts->cp_input_info[i].input.  Thus on jumping to fail, it will not be
freed.  In this case, we want to free, but not unregister, so the free for
this most recently allocated resource is put before the jump.

A simplified version of the semantic match that finds this problem is:
(http://coccinelle.lip6.fr/)

// <smpl>
@r exists@
local idexpression struct input_dev * x;
expression ra,rr;
position p1,p2;
@@

x = input_allocate_device@p1(...)
...  when != x = rr
     when != input_free_device(x,...)
     when != if (...) { ... input_free_device(x,...) ...}
if(...) { ... when != x = ra
     when forall
     when != input_free_device(x,...)
 \(return <+...x...+>; \| return@p2...; \) }

@script:python@
p1 << r.p1;
p2 << r.p2;
@@

cocci.print_main("input_allocate_device",p1)
cocci.print_secs("input_free_device",p2)
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/cptm1217/clearpad_tm1217.c