mirror of
https://github.com/torvalds/linux.git
synced 2026-07-16 18:45:58 +02:00
1790c228fb
This code does not call deinit_card(card); in an error case, as done in other error-handling code in the same function. But actually, the called function init_sram can only return 0, so there is no need for the error check at all. init_sram is also given a void return type, and its single return statement at the end of the function is dropped. A simplified version of the sematic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r exists@ @r@ statement S1,S2,S3; constant C1,C2,C3; @@ *if (...) {... S1 return -C1;} ... *if (...) {... when != S1 return -C2;} ... *if (...) {... S1 return -C3;} // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: David S. Miller <davem@davemloft.net> |
||
|---|---|---|
| .. | ||
| .gitignore | ||
| adummy.c | ||
| ambassador.c | ||
| ambassador.h | ||
| atmtcp.c | ||
| eni.c | ||
| eni.h | ||
| firestream.c | ||
| firestream.h | ||
| fore200e.c | ||
| fore200e.h | ||
| he.c | ||
| he.h | ||
| horizon.c | ||
| horizon.h | ||
| idt77105.c | ||
| idt77105.h | ||
| idt77252_tables.h | ||
| idt77252.c | ||
| idt77252.h | ||
| iphase.c | ||
| iphase.h | ||
| Kconfig | ||
| lanai.c | ||
| Makefile | ||
| midway.h | ||
| nicstar.c | ||
| nicstar.h | ||
| nicstarmac.c | ||
| nicstarmac.copyright | ||
| solos-attrlist.c | ||
| solos-pci.c | ||
| suni.c | ||
| suni.h | ||
| tonga.h | ||
| uPD98401.h | ||
| uPD98402.c | ||
| uPD98402.h | ||
| zatm.c | ||
| zatm.h | ||
| zeprom.h | ||