Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

socket: fix leaking tcp_connection_contexts and opa objects #214

Merged
merged 3 commits into from
Apr 17, 2024
Merged

socket: fix leaking tcp_connection_contexts and opa objects #214

merged 3 commits into from
Apr 17, 2024

Conversation

bonifaido
Copy link
Member

@bonifaido bonifaido commented Apr 17, 2024 •
edited
Loading

Description

The title is self-explanatory.

Type of Change

  • Bug Fix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@bonifaido bonifaido changed the title socket: fix leaking tcp_connection_contexts socket: fix leaking tcp_connection_contexts and opa objects Apr 17, 2024
@bonifaido bonifaido self-assigned this Apr 17, 2024
@bonifaido bonifaido marked this pull request as ready for review April 17, 2024 13:48
@bonifaido bonifaido requested a review from a team as a code owner April 17, 2024 13:48
baluchicken
baluchicken reviewed Apr 17, 2024
View reviewed changes
src/socket.c Show resolved Hide resolved
src/socket.c Show resolved Hide resolved
@bonifaido bonifaido merged commit 5f76332 into main Apr 17, 2024
5 checks passed
@bonifaido bonifaido deleted the tcp_connection_context_free branch April 17, 2024 15:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@baluchicken baluchicken baluchicken approved these changes

Assignees

@bonifaido bonifaido

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bonifaido @baluchicken