Skip to content
New issue

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

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

Already on GitHub? # to your account

bugfix: SrsAutoFree addrinfo #2886

Merged
merged 2 commits into from
Jan 22, 2022
Merged

bugfix: SrsAutoFree addrinfo #2886

merged 2 commits into from
Jan 22, 2022

Conversation

bluestn
Copy link
Contributor

@bluestn bluestn commented Jan 22, 2022

bugfix: getaddrinfo return cannot freed by delete

@codecov-commenter
Copy link

codecov-commenter commented Jan 22, 2022

Codecov Report

Merging #2886 (c3b97dc) into 4.0release (dc43a11) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff               @@
##           4.0release    #2886      +/-   ##
==============================================
- Coverage       60.22%   60.21%   -0.01%     
==============================================
  Files             121      121              
  Lines           51076    51078       +2     
==============================================
  Hits            30758    30758              
- Misses          20318    20320       +2     

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

'| Impacted Files | Coverage Δ | |
|---|---|---|
| trunk/src/kernel/srs_kernel_utility.cpp | 92.37% <100.00%> (ø) | |
| trunk/src/protocol/srs_service_st.cpp | 78.38% <100.00%> (ø) | |'

Translated to English while maintaining the markdown structure:

'| trunk/src/kernel/srs_kernel_utility.cpp | 92.37% <100.00%> (ø) | |
| trunk/src/protocol/srs_service_st.cpp | 78.38% <100.00%> (ø) | |
| trunk/src/app/srs_app_source.cpp | 0.66% <0.00%> (-0.01%) | ⬇️ |
| trunk/src/core/srs_core_autofree.hpp | 93.33% <0.00%> (ø) | |

'

Translated to English while maintaining the markdown structure:

'| trunk/src/core/srs_core_autofree.hpp | 93.33% <0.00%> (ø) | |


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data'

Translated to English while maintaining the markdown structure:

'> Δ = absolute <relative> (impact), ø = not affected, ? = missing data

Powered by Codecov. Last update dc43a11...c3b97dc. Read the comment docs.

TRANS_BY_GPT3

@winlinvip
Copy link
Member

@bluestn Good job! This patch is simple and stable enough. 👍

@winlinvip winlinvip merged commit 386ac44 into ossrs:4.0release Jan 22, 2022
winlinvip added a commit that referenced this pull request Jan 23, 2022
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants