Project

General

Profile

Actions

Bug #9856

closed

[PATCH] process.c (proc_getgroups, proc_setgroups): use ALLOCV_N

Added by normalperson (Eric Wong) almost 10 years ago. Updated over 9 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 2.2.0dev (2014-05-19 trunk 46010) [x86_64-linux]
[ruby-core:62694]

Description

On my Linux system, these may return up to 64K rb_gid_t entries for
a total of 256K stack usage, potentially triggering SystemStackError
or just a segfault. Use ALLOCV_N for stack safety, especially since
these are not performance-critical methods.

Will commit in a day or two.

http://bogomips.org/ruby.git/patch?id=9074e3808d


Files

process-allocv_n.patch (1.92 KB) process-allocv_n.patch normalperson (Eric Wong), 05/20/2014 02:17 AM
Actions #1

Updated by Anonymous almost 10 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

Applied in changeset r46075.


process.c (proc_getgroups, proc_setgroups): use ALLOCV_N

  • process.c (proc_getgroups, proc_setgroups): use ALLOCV_N
    [Bug #9856]

Updated by usa (Usaku NAKAMURA) almost 10 years ago

  • Backport changed from 2.0.0: REQUIRED, 2.1: REQUIRED to 2.0.0: DONE, 2.1: REQUIRED

backported into ruby_2_0_0 at r46581.

Updated by nagachika (Tomoyuki Chikanaga) over 9 years ago

  • Backport changed from 2.0.0: DONE, 2.1: REQUIRED to 2.0.0: DONE, 2.1: DONE

Backported into ruby_2_1 branch at r46615.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0