diff options
| author | Amin Hassani <ahassani@chromium.org> | 2018-07-26 11:19:10 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-07-27 12:12:48 -0700 |
| commit | abe4a775df7e9b7b8f09094988f6f1b42acc05a3 (patch) | |
| tree | 0a9e5e08d3a0d8ee5a0e75c1fff529723113e43c /update_manager/update_time_restrictions_policy_impl.h | |
| parent | 0dbf1f9b946a11a76116322d5603d0f11a206dc5 (diff) | |
update_engine: Remove Action object from FilesystemVerifierActionDelegate
in CL:1065113 we forgot to remove the FileSystemVerfierAction instance from the
FilesystemVerifierActionDelegate. Hence, it will be removed from memory while
FilesystemVerifierActionDelegate still keeps a pointer to it and access it. This
causes a possible race condition on ExitMainLoop() function. This patch fixes
the issue by removing that instance and the unnecessary checks for the
cleanup. Once the Action is deleted by the ActionProcessor it will be cleaned up
(which will happen when the action is terminated too). So there is not really a
need for testing the cleanup method.
BUG=chromium:867815
TEST=unittests pass
Change-Id: Id9d3f361eb916007e95dbb1d0adb9603ceae00b9
Reviewed-on: https://chromium-review.googlesource.com/1151902
Commit-Ready: Lann Martin <lannm@chromium.org>
Tested-by: Amin Hassani <ahassani@chromium.org>
Reviewed-by: Sen Jiang <senj@chromium.org>
Diffstat (limited to 'update_manager/update_time_restrictions_policy_impl.h')
0 files changed, 0 insertions, 0 deletions