diff options
| author | Nicolas Lacasse <nlacasse@google.com> | 2018-11-07 13:54:47 -0800 |
|---|---|---|
| committer | Shentubot <shentubot@google.com> | 2018-11-07 13:55:36 -0800 |
| commit | 13b48f2e6a186321084fa8159e8cc2659ed221a2 (patch) | |
| tree | 8bf0f439df3781468734de92f306391b8c9e8955 /runsc/container/container_test.go | |
| parent | c92b9b7086b89fd8e7f5913bf74d04761163e24b (diff) | |
AsyncBarrier should be run after all defers in destroyContainerFS.
destroyContainerFS must wait for all async operations to finish before
returning. In an attempt to do this, we call fs.AsyncBarrier() at the end of
the function. However, there are many defer'd DecRefs which end up running
AFTER the AsyncBarrier() call.
This CL fixes this by calling fs.AsyncBarrier() in the first defer statement,
thus ensuring that it runs at the end of the function, after all other defers.
PiperOrigin-RevId: 220523545
Change-Id: I5e96ee9ea6d86eeab788ff964484c50ef7f64a2f
Diffstat (limited to 'runsc/container/container_test.go')
0 files changed, 0 insertions, 0 deletions