diff options
| author | Alexander Usyskin <alexander.usyskin@intel.com> | 2014-01-27 20:27:23 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-02-22 21:32:27 (GMT) |
| commit | cb2a6e8a6e423b1424fdd792640943a53a3d70f3 (patch) | |
| tree | ef79f2ddd7c92834ef617a206745da5d188443a9 /drivers/misc | |
| parent | 92cf5e3789ac4f7ee72029b4df1451966a83a43c (diff) | |
| download | linux-fsl-qoriq-cb2a6e8a6e423b1424fdd792640943a53a3d70f3.tar.xz | |
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 e0684b4..6a30306 100644 --- a/drivers/misc/mei/client.c +++ b/drivers/misc/mei/client.c @@ -916,8 +916,16 @@ void mei_cl_all_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); } |