diff options
| author | Alexander Usyskin <alexander.usyskin@intel.com> | 2014-01-27 22:27:23 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-02-22 12:41:28 -0800 |
| commit | 85153361d846360f4e94e187a04913ad063348a2 (patch) | |
| tree | 6de208bbb6d9ab00f3456dff5e0e3c9e42d258c0 /drivers/misc | |
| parent | 2fff6c25c0a8e2df0bcb2f578b892356b1fe4c80 (diff) | |
| download | lwn-85153361d846360f4e94e187a04913ad063348a2.tar.gz lwn-85153361d846360f4e94e187a04913ad063348a2.zip | |
mei: clear write cb from waiting list on reset
commit 30c54df7cb9b15b222529a028390b9c9582dd65e upstream.
Clear write callbacks sitting in write_waiting list on reset.
Otherwise these callbacks are left dangling and cause memory leak.
Signed-off-by: Alexander Usyskin <alexander.usyskin@intel.com>
Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/misc')
| -rw-r--r-- | drivers/misc/mei/client.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/drivers/misc/mei/client.c b/drivers/misc/mei/client.c index e310ca6ed1a3..bdc2cca7ed4f 100644 --- a/drivers/misc/mei/client.c +++ b/drivers/misc/mei/client.c @@ -829,8 +829,16 @@ void mei_cl_all_read_wakeup(struct mei_device *dev) void mei_cl_all_write_clear(struct mei_device *dev) { struct mei_cl_cb *cb, *next; + struct list_head *list; - list_for_each_entry_safe(cb, next, &dev->write_list.list, list) { + list = &dev->write_list.list; + list_for_each_entry_safe(cb, next, list, list) { + list_del(&cb->list); + mei_io_cb_free(cb); + } + + list = &dev->write_waiting_list.list; + list_for_each_entry_safe(cb, next, list, list) { list_del(&cb->list); mei_io_cb_free(cb); } |