Skip to content

Manage unpooled buffer lifecycle - #80

Merged
Z-Kris merged 6 commits into
blurite:masterfrom
Khaled431:rsprot-unpooled-buffer-leaks
Sep 3, 2026
Merged

Manage unpooled buffer lifecycle#80
Z-Kris merged 6 commits into
blurite:masterfrom
Khaled431:rsprot-unpooled-buffer-leaks

Conversation

@Khaled431

@Khaled431 Khaled431 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • make binary blobs explicitly closeable while retaining a cleaner fallback for abandoned streams
  • release login, reconnect, and failure buffers
  • avoid reference-counted direct buffers when possible

Verification

  • compiled and linted the buffer, protocol, and HAProxy modules
  • compiled and linted OSRS 221-240 model and API modules
  • completed two independent buffer-ownership reviews
  • ran locally and validated the behavior and issue cases are resolved

@Khaled431
Khaled431 force-pushed the rsprot-unpooled-buffer-leaks branch from 58f044e to 206d567 Compare September 3, 2026 02:59
@Khaled431
Khaled431 marked this pull request as ready for review September 3, 2026 03:28
@Z-Kris
Z-Kris merged commit a42b0e7 into blurite:master Sep 3, 2026
3 checks passed
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