Bug #10923: Bug in io/wait/wait.c

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: trunk, v2.2.x
• Backport: 2.0.0: REQUIRED, 2.1: DONE, 2.2: DONE

[funny_falcon (Yura Sokolov)]

https://github.com/ruby/ruby/blob/28b9f11d5a4b878adb50a3e5472ceea8c462ed4a/ext/io/wait/wait.c#L64-L65
return value should be wrapped by INT2FIX

[normalperson (Eric Wong)]

• Backport changed from 2.0.0: UNKNOWN, 2.1: UNKNOWN, 2.2: UNKNOWN to 2.0.0: REQUIRED, 2.1: REQUIRED, 2.2: REQUIRED

Fixed in r49797

[nagachika (Tomoyuki Chikanaga)]

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

Applied in changeset ruby-21:r49812.

---

merge revision(s) r49797: [Backport #10923]

* ext/io/wait/wait.c (io_nread): wrap return value with INT2FIX
  Thanks to Yura Sokolov <[e-mail]>
  [ruby-core:68369] [Bug#10923]

* test/io/wait/test_io_wait.rb (test_nread_buffered):
  fix broken test

[nagachika (Tomoyuki Chikanaga)]

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

Backported into ruby_2_1 branch at r49812.

[nagachika (Tomoyuki Chikanaga)]

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

Backported into ruby_2_2 branch at r50276.