Forum: Ruby-core [Open] DateTime#jisx0301 returns corrupted string when GC.stress is set to true.

Ba7345a85653fd9c1022361df622abfb?d=identicon&s=25 unknown (Guest)
on 2014-08-17 19:02
(Received via mailing list)
Issue #10146 has been reported by Masamitsu Murase.

----------------------------------------
Bug #10146: DateTime#jisx0301 returns corrupted string when GC.stress is
set to true.
https://bugs.ruby-lang.org/issues/10146

* Author: Masamitsu Murase
* Status: Open
* Priority: Normal
* Assignee:
* Category:
* Target version:
* ruby -v: ruby 2.2.0dev (2014-08-17) [i386-mswin32_100]
* Backport: 2.0.0: UNKNOWN, 2.1: UNKNOWN
----------------------------------------
DateTime#jisx0301 returns corrupted string when GC.stress is set to
true.

**Sample script to reproduce this issue:**

~~~
# sample.rb
require("date")

GC.stress = true
p DateTime.parse('2001-02-03T04:05:06.123456789+07:00').jisx0301(9)
~~~

This script shows invalid string as follows:

~~~
> ruby sample.rb
"\x16T04:05:06.123456789+07:00"
~~~

The following string is expected:

~~~
"H13.02.03T04:05:06.123456789+07:00"
~~~


**Cause:**
This is caused by unexpected GC.
Some variables in 'ext/date/date_core.c' should be marked with
RB_GC_GUARD to prevent GC.

For example, variable 's' in dt_lite_jisx0301 should be marked with
RB_GC_GUARD
so that the object pointed by 's' is kept while rb_str_append is called.

~~~
static VALUE
dt_lite_jisx0301(int argc, VALUE *argv, VALUE self)
{
    VALUE n, s;
/* ...snip... */
    {
  get_d1(self);
  s = jisx0301_date(m_real_local_jd(dat),
        m_real_year(dat));
  return rb_str_append(strftimev(RSTRING_PTR(s), self, set_tmx),
           iso8601_timediv(self, n));
    }
}
~~~

should be

~~~
static VALUE
dt_lite_jisx0301(int argc, VALUE *argv, VALUE self)
{
    VALUE n, s, result;
/* ...snip... */
    {
  get_d1(self);
  s = jisx0301_date(m_real_local_jd(dat),
        m_real_year(dat));
  result = rb_str_append(strftimev(RSTRING_PTR(s), self, set_tmx),
             iso8601_timediv(self, n));
  RB_GC_GUARD(s);
  return result;
    }
}
~~~


I attached a patch, date_core.c.diff, which also includes other fixes
regarding RB_GC_GUARD in date_core.c.
If this patch is not so appropriate, please ignore it.

Regards,
Murase


---Files--------------------------------
date_core.c.diff (1.72 KB)
This topic is locked and can not be replied to.