diff options
| author | Julia Lawall <julia@diku.dk> | 2008-06-10 10:03:25 +0200 |
|---|---|---|
| committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2008-06-10 10:03:28 +0200 |
| commit | 1783e60ff207805a3e75cf522b17ec9bb1604a62 (patch) | |
| tree | 8245a54df59222e7e8ecd132c13f6c133284adfb /drivers/s390 | |
| parent | b57838ea3f860c97e4726ab93abdf83a3d079a66 (diff) | |
| download | kernel_samsung_aries-1783e60ff207805a3e75cf522b17ec9bb1604a62.zip kernel_samsung_aries-1783e60ff207805a3e75cf522b17ec9bb1604a62.tar.gz kernel_samsung_aries-1783e60ff207805a3e75cf522b17ec9bb1604a62.tar.bz2 | |
[S390] tape_3590.c: introduce missing kfree
The semantic match that finds the problem is as follows:
(http://www.emn.fr/x-info/coccinelle/)
@r exists@
expression E,E1;
statement S;
position p1,p2,p3;
@@
E =@p1 \(kmalloc\|kcalloc\|kzalloc\)(...)
... when != E = E1
if (E == NULL || ...) S
... when != E = E1
if@p2 (...) {
... when != kfree(E)
}
... when != E = E1
kfree@p3(E);
@forall@
position r.p2;
expression r.E;
int E1 != 0;
@@
* if@p2 (...) {
... when != kfree(E)
when strict
return E1; }
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers/s390')
| -rw-r--r-- | drivers/s390/char/tape_3590.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/s390/char/tape_3590.c b/drivers/s390/char/tape_3590.c index 8246ef3..42ce791 100644 --- a/drivers/s390/char/tape_3590.c +++ b/drivers/s390/char/tape_3590.c @@ -1598,7 +1598,7 @@ tape_3590_setup_device(struct tape_device *device) rc = tape_3590_read_dev_chars(device, rdc_data); if (rc) { DBF_LH(3, "Read device characteristics failed!\n"); - goto fail_kmalloc; + goto fail_rdc_data; } rc = tape_std_assign(device); if (rc) |