Skip to content

fix(tls): duplicated buffer init#381

Merged
Berrysoft merged 1 commit intocompio-rs:masterfrom
incisakura:duplicated-buffer-init
Mar 9, 2025
Merged

fix(tls): duplicated buffer init#381
Berrysoft merged 1 commit intocompio-rs:masterfrom
incisakura:duplicated-buffer-init

Conversation

@incisakura
Copy link
Contributor

slice.fill(MaybeUninit::new(0)) will be called every time read_buf() is invoked, even if the buffer has been initialized previously.

Copy link
Member

@Berrysoft Berrysoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@incisakura incisakura force-pushed the duplicated-buffer-init branch from 79e029e to 23b8391 Compare March 9, 2025 07:13
@Berrysoft Berrysoft force-pushed the duplicated-buffer-init branch from 23b8391 to 9562fc8 Compare March 9, 2025 07:23
@Berrysoft Berrysoft merged commit 0704ae0 into compio-rs:master Mar 9, 2025
48 checks passed
@incisakura
Copy link
Contributor Author

Thank you!!!

George-Miao pushed a commit to George-Miao/compio that referenced this pull request May 27, 2025
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