diff options
| author | Julia Lawall <julia@diku.dk> | 2009-07-12 09:42:47 +0200 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.cz> | 2009-07-23 01:28:02 +0200 |
| commit | 3040c8203d68a9a0564af81729085054fd6b5b03 (patch) | |
| tree | 5a1036cf9c9fce40f2b9cfb369745fbabb09c987 /drivers/hid | |
| parent | 711a680e35059bc5c7c28d3c4bd0bebd3b7bb6ee (diff) | |
HID: Drop NULL test on list_entry result
list_entry, which is an alias for container_of, cannot return NULL, as
there is no way to add a NULL value to a doubly linked list.
A simplified version of the semantic match that findds this problem is as
follows:
(http://www.emn.fr/x-info/coccinelle/)
// <smpl>
@r@
expression x,E;
statement S1,S2;
position p,p1;
@@
*x = list_entry@p(...)
... when != x = E
*if@p1 (x == NULL) S1 else S2
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid')
| -rw-r--r-- | drivers/hid/hid-lgff.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/hid/hid-lgff.c b/drivers/hid/hid-lgff.c index 95835ea5690..987abebe082 100644 --- a/drivers/hid/hid-lgff.c +++ b/drivers/hid/hid-lgff.c @@ -151,11 +151,6 @@ int lgff_init(struct hid_device* hid) /* Check that the report looks ok */ report = list_entry(report_list->next, struct hid_report, list); - if (!report) { - err_hid("NULL output report"); - return -1; - } - field = report->field[0]; if (!field) { err_hid("NULL field"); |