commit | 667b6b022afb8c4b0caece2b105b15ffee290cba | [log] [tgz] |
---|---|---|
author | Geoff Maddox <gmaddox@lucidchart.com> | Tue Sep 04 04:45:00 2018 -0700 |
committer | Copybara-Service <copybara-piper@google.com> | Tue Sep 04 04:47:04 2018 -0700 |
tree | 380fa1e3c71d6cf961bc98d1915eddbb26bedf94 | |
parent | 2613b6244bc8987bc9937a0635fce8820df5ff61 [diff] |
Fix race condition with releaseUploadChannel(). A race condition can occur in `HttpBlobStore.releaseUploadChannel()` when `ChannelHandlerContext` is closed by `HttpUploadHandler.channelRead0`. Because `ChannelHandlerContext.close` runs asynchronously and closes all pipelines in reverse order (before the channel is marked as closed), `HttpBlobStore.releaseUploadChannel()` can be executed while channel is still open and handlers are being removed, causing an exception to be thrown by the `ch.pipeline().remove(X)` calls. This fix saves the status of the keepAlive response from the `HttpUploadHandler` so that the channel can be fully closed before releasing. Resolves #5952 Closes #5953. PiperOrigin-RevId: 211437717
{Fast, Correct} - Choose two
Build and test software of any size, quickly and reliably.
Speed up your builds and tests: Bazel only rebuilds what is necessary. With advanced local and distributed caching, optimized dependency analysis and parallel execution, you get fast and incremental builds.
One tool, multiple languages: Build and test Java, C++, Android, iOS, Go, and a wide variety of other language platforms. Bazel runs on Windows, macOS, and Linux.
Scalable: Bazel helps you scale your organization, codebase, and continuous integration solution. It handles codebases of any size, in multiple repositories or a huge monorepo.
Extensible to your needs: Easily add support for new languages and platforms with Bazel's familiar extension language. Share and re-use language rules written by the growing Bazel community.
Follow our tutorials:
See CONTRIBUTING.md
Bazel is released in ‘Beta’. See the product roadmap to learn about the path toward a stable 1.0 release.