Skip to content

Fix Perl async callback refcounts#35

Open
xintenseapple wants to merge 1 commit into
freenginx:defaultfrom
xintenseapple:fix/issue-26-perl-async-callback-refcnt
Open

Fix Perl async callback refcounts#35
xintenseapple wants to merge 1 commit into
freenginx:defaultfrom
xintenseapple:fix/issue-26-perl-async-callback-refcnt

Conversation

@xintenseapple

Copy link
Copy Markdown

Summary:

  • Validate async Perl callbacks as CODE refs, hold an owned Perl reference while they are pending, and release it on dispatch, setup failure, or request cleanup.

Verification:

  • git diff --check; ./auto/configure --with-http_perl_module --without-http_rewrite_module; Perl XS and ngx_http_perl_module.c compiled; final link fails in this environment because libperl is missing

Fixes #26

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 Perl async callbacks store unrefcounted SV/CV pointers

1 participant