Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix: correct offset vector memory allocation size for PCRE2. #341

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

zhongweiy
Copy link
Contributor

Also update the name of ovecsize in ngx_http_lua_ffi_exec_regex to reflect the difference.

@zhongweiy
Copy link
Contributor Author

This PR is a follow-up for openresty/lua-nginx-module#2290.

bungle added a commit to Kong/kong that referenced this pull request Mar 7, 2024
### Summary

This adds two commits from @zhongweiy as found here:
openresty/lua-nginx-module#2291
openresty/stream-lua-nginx-module#341

They are cleanups to original patch.

Signed-off-by: Aapo Talvensaari <[email protected]>
@zhuizhuhaomeng zhuizhuhaomeng merged commit a5c924b into openresty:master Mar 8, 2024
3 checks passed
bungle added a commit to Kong/kong that referenced this pull request Mar 13, 2024
### Summary

This adds two commits from @zhongweiy as found here:
openresty/lua-nginx-module#2291
openresty/stream-lua-nginx-module#341

They are cleanups to original patch.

Signed-off-by: Aapo Talvensaari <[email protected]>
github-actions bot pushed a commit to Kong/kong that referenced this pull request Mar 13, 2024
### Summary

This adds two commits from @zhongweiy as found here:
openresty/lua-nginx-module#2291
openresty/stream-lua-nginx-module#341

They are cleanups to original patch.

Signed-off-by: Aapo Talvensaari <[email protected]>
(cherry picked from commit 5fea640)
bungle added a commit to Kong/kong that referenced this pull request Mar 13, 2024
### Summary

This adds two commits from @zhongweiy as found here:
openresty/lua-nginx-module#2291
openresty/stream-lua-nginx-module#341

They are cleanups to original patch.

Signed-off-by: Aapo Talvensaari <[email protected]>
(cherry picked from commit 5fea640)
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.

2 participants