diff options
| author |   Amitoj Kaur Chawla <amitoj1606@gmail.com> | 2016-03-12 00:11:47 +0530 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-03-11 22:09:09 -0800 |
| commit | 720f7a7850556f8d9ed6069cbcfd9b34b105e04a (patch) | |
| tree | f09b01316d2755f70f7661c7b26f3739ecb52ea0 /drivers/staging/most/hdm-usb/hdm_usb.c | |
| parent | staging: rts5208: Change form of NULL comparisons (diff) | |
| download | linux-dev-720f7a7850556f8d9ed6069cbcfd9b34b105e04a.tar.xz linux-dev-720f7a7850556f8d9ed6069cbcfd9b34b105e04a.zip | |
staging: most: hdm-usb: Remove invalid reference error
commit e3479f77("staging: most: hdm-usb: Remove create_workqueue()")
cancel_work_sync(&anchor->clear_work_obj) is introduced after freeing
`anchor` causing a invalid reference error. This patch removes this
error by shifting the call to cancel_work_sync before freeing
`anchor`.
Signed-off-by: Amitoj Kaur Chawla <amitoj1606@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/most/hdm-usb/hdm_usb.c')
| -rw-r--r-- | drivers/staging/most/hdm-usb/hdm_usb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/staging/most/hdm-usb/hdm_usb.c b/drivers/staging/most/hdm-usb/hdm_usb.c index 78031abaf857..aeae071f2823 100644 --- a/drivers/staging/most/hdm-usb/hdm_usb.c +++ b/drivers/staging/most/hdm-usb/hdm_usb.c @@ -221,11 +221,10 @@ static void free_anchored_buffers(struct most_dev *mdev, unsigned int channel) } spin_lock_irqsave(&mdev->anchor_list_lock[channel], flags); list_del(&anchor->list); + cancel_work_sync(&anchor->clear_work_obj); kfree(anchor); } spin_unlock_irqrestore(&mdev->anchor_list_lock[channel], flags); - - cancel_work_sync(&anchor->clear_work_obj); } /** |