Skip to content

fix: harden connection close and protocol reads - #510

Open
HashimTheArab wants to merge 1 commit into
Sandertv:masterfrom
HashimTheArab:fix/connection-reader-safety-upstream
Open

fix: harden connection close and protocol reads#510
HashimTheArab wants to merge 1 commit into
Sandertv:masterfrom
HashimTheArab:fix/connection-reader-safety-upstream

Conversation

@HashimTheArab

Copy link
Copy Markdown
Contributor

What changed

  • avoid nil dereferences when closing a partially initialized connection
  • use io.ReadFull for length-prefixed strings and byte slices so empty terminal values and partial reads are handled correctly

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.

1 participant