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

Move resource_user ownership into chttp2 transport/server/connector v2 #27032

Merged
merged 5 commits into from
Aug 19, 2021

Conversation

drfloob
Copy link
Member

@drfloob drfloob commented Aug 17, 2021

Reintroducing PR #26643, which was reverted in #27029

Fixed a memory leak and added a test that would have caught it (ASAN build): ca0c8c4

@drfloob drfloob added area/core lang/core release notes: no Indicates if PR should not be in release notes labels Aug 17, 2021
@drfloob
Copy link
Member Author

drfloob commented Aug 18, 2021

I've fixed the iOS build issue separately: #27055.

@drfloob drfloob merged commit d10617e into grpc:master Aug 19, 2021
TaWeiTu added a commit to TaWeiTu/grpc that referenced this pull request Aug 19, 2021
TaWeiTu added a commit to TaWeiTu/grpc that referenced this pull request Aug 19, 2021
Vignesh2208 pushed a commit to Vignesh2208/grpc that referenced this pull request Aug 20, 2021
grpc#27032)

Reintroducing PR grpc#26643, which was reverted in grpc#27029

Fixed a memory leak and added a test that would have caught it (ASAN build): ca0c8c4
TaWeiTu added a commit to TaWeiTu/grpc that referenced this pull request Aug 20, 2021
TaWeiTu added a commit to TaWeiTu/grpc that referenced this pull request Aug 20, 2021
ctiller added a commit to ctiller/grpc that referenced this pull request Aug 30, 2021
@drfloob drfloob deleted the revert-f5d3ed branch September 1, 2021 16:45
lidizheng pushed a commit to lidizheng/grpc that referenced this pull request Sep 23, 2021
grpc#27032)

Reintroducing PR grpc#26643, which was reverted in grpc#27029

Fixed a memory leak and added a test that would have caught it (ASAN build): ca0c8c4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core lang/core release notes: no Indicates if PR should not be in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants