Skip to content

Guard active threaded sendfile task context#31

Open
xintenseapple wants to merge 1 commit into
freenginx:defaultfrom
xintenseapple:fix/issue-22-h2-threaded-sendfile-reentry
Open

Guard active threaded sendfile task context#31
xintenseapple wants to merge 1 commit into
freenginx:defaultfrom
xintenseapple:fix/issue-22-h2-threaded-sendfile-reentry

Conversation

@xintenseapple

Copy link
Copy Markdown

Summary:

  • Return NGX_DONE for an already-active threaded sendfile task before overwriting the shared connection task context.

Verification:

  • git diff --check; ./auto/configure --builddir=/tmp/freenginx-issue-22-build --with-threads --with-http_v2_module --without-http_rewrite_module --without-http_gzip_module --without-pcre; make -f /tmp/freenginx-issue-22-build/Makefile -j2

Fixes #22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP/2 threaded sendfile reentry races shared connection task context

1 participant