Bug #10062: Segfault involving Queue

• Status: Closed
• Assignee: -
• Target version: -
• ruby -v: trun
• Backport: 2.0.0: DONTNEED, 2.1: DONE

[LordAlveric (Fred Mitchell)]

Pushing an object onto a Queue instantiation causes a segfault. This was seen on Ruby versions 2.1.1 and 2.1.2

In the interests of being able to completely reproduce the environment in which this happens, I have included detailed instructions here.

1. Clone https://github.com/flajann2/rubyneat/tree/devready (yes, the devready branch)
2. In its root do "rake install" to install the gem
3. Clone https://github.com/flajann2/rubyneat_dashboard/tree/master
4. In its root do "rake install" to install that gem
5. Clone https://github.com/flajann2/rubyneat_examples/tree/devready (yes, the devready branch)
6. Go to the root of that and
7. Run "bundle update"
8. Run "neat run xor"
9. It should segfault almost immediately.

[LordAlveric (Fred Mitchell)]

The condition has been tagged as 'segfault' on all three projects. Since I will attempt to find a workaround, be sure to pull those tags, as well as 'devready' branches on rubyneat and rubyneat_examples

[nagachika (Tomoyuki Chikanaga)]

Hello fred, thank you for your reporting.

At first, could you try to reproduce it with 2.1-head and trunk?

[nagachika (Tomoyuki Chikanaga)]

• Status changed from Open to Feedback
• Priority changed from 7 to Normal

Does anyone can reproduce this?

[nobu (Nobuyoshi Nakada)]

• File bug-10062.txt bug-10062.txt added
• Description updated (diff)

Seems the Queue instance is not initialized properly.
Probably, Queue#initialize is overridden somewhere?

$ ruby -rthread -e 'class Thread::Queue; def initialize;end;end;q=Thread::Queue.new;q.push(nil)'
-e:1: [BUG] Segmentation fault at 0x00000000000008
ruby 2.2.0dev (2014-08-18 trunk 47214) [universal.x86_64-darwin13]

[nobu (Nobuyoshi Nakada)]

• Tracker changed from Backport to Bug
• Project changed from Backport21 to Ruby master
• ruby -v set to trun
• Backport set to 2.0.0: REQUIRED, 2.1: REQUIRED

[nobu (Nobuyoshi Nakada)]

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

Applied in changeset r47217.

---

thread.c: check initialized

• ext/thread/thread.c (get_array): check instance variables are
  initialized properly. [ruby-core:63826][Bug #10062]

[nagachika (Tomoyuki Chikanaga)]

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

Backported into ruby_2_1 branch at r47378.

[usa (Usaku NAKAMURA)]

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