Project

General

Profile

Actions

Bug #11123

closed

[PATCH] bsock_recmvsg_internal needs GC guard for control buffer

Added by normalperson (Eric Wong) about 7 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Target version:
-
[ruby-core:69082]

Description

The control buffer may be used throughout the function, so
prevent the string from being lost to GC.

I'm pretty sure this is correct and we'll need it, I'll commit in a day or two
unless I missed something. I was planning to introduce new features before
I noticed this bug:

  • "exception: false"
  • explicit buffer reuse to reduce GC, similar to read_nonblock(num, str)

I also feel 4K buffers are too big for stack and should use heap instead
(as ALLOCV caps stack allocations at 1K)


Files

Actions

Also available in: Atom PDF