Backport #8101: Backport fix for time.send(:initialize) reinitializing and modifying value

• Status: Closed
• Assignee: nagachika (Tomoyuki Chikanaga)

[headius (Charles Nutter)]

I filed https://bugs.ruby-lang.org/issues/8099 to get Time to prevent reinitialization, and nobu fixed it in r39766. I am requesting that it be backported to 1.9.3 along with tests I will be adding shortly.

[headius (Charles Nutter)]

Nevermind about additional tests...I see nobu did add one. So I'd just like to backport that commit to 1.9.3.

[usa (Usaku NAKAMURA)]

• Status changed from Open to Assigned
• Assignee set to usa (Usaku NAKAMURA)

[usa (Usaku NAKAMURA)]

• Project changed from 13 to 15
• Assignee changed from usa (Usaku NAKAMURA) to nagachika (Tomoyuki Chikanaga)

Seems to be backported to 2.0.0 first.

[nagachika (Tomoyuki Chikanaga)]

Sorry for my late response. I'll merge r39766 and r39769. r39769 delete broken test by denial of re-initialize Time.

[nagachika (Tomoyuki Chikanaga)]

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

This issue was solved with changeset r40282.
Charles, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

merge revision(s) 39766,39769: [Backport #8101]

* time.c (GetTimeval): check if already initialized instance.

* time.c (GetNewTimeval): check if newly created instance.

* time.c (time_init_0, time_init_1, time_init_copy, time_mload): must
  be newly created instance.  [ruby-core:53436] [Bug #8099]

[nagachika (Tomoyuki Chikanaga)]

• Project changed from 15 to 13
• Status changed from Closed to Assigned
• Assignee changed from nagachika (Tomoyuki Chikanaga) to usa (Usaku NAKAMURA)

[usa (Usaku NAKAMURA)]

• Status changed from Assigned to Closed

This issue was solved with changeset r40711.
Charles, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

merge revision(s) 39766,39769: [Backport #8101]

* time.c (GetTimeval): check if already initialized instance.

* time.c (GetNewTimeval): check if newly created instance.

* time.c (time_init_0, time_init_1, time_init_copy, time_mload): must
  be newly created instance.  [ruby-core:53436] [Bug #8099]

[ktsj (Kazuki Tsujimoto)]

• Project changed from 13 to 15
• Status changed from Closed to Assigned
• Assignee changed from usa (Usaku NAKAMURA) to nagachika (Tomoyuki Chikanaga)

r42456 is related to r39766, so please backport it, too.

[nagachika (Tomoyuki Chikanaga)]

• Status changed from Assigned to Closed

Applied in changeset r44844.

---

merge revision(s) r42456: [Backport #8101]

* time.c (get_timeval, get_new_timeval): use rb_obj_class()
  instead of CLASS_OF() because CLASS_OF() may return
  a singleton class.