diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-06-16 09:46:06 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-06-16 09:46:06 +0000 |
commit | e4d0f54b1b0e3391eabbb961530beaffcbc7879a (patch) | |
tree | aaf2fb9614656765de968cae29bdbffc76940a7c /ext/date | |
parent | d1496e8b8cfd3554a4ce72c33d42e4cb33667e96 (diff) | |
download | ruby-e4d0f54b1b0e3391eabbb961530beaffcbc7879a.tar.gz |
date_parse.c: use ALLOCV
* ext/date/date_parse.c (s3e, date_zone_to_diff, parse_ddd_cb):
use ALLOCV instead of ALLOCA get rid of stack overflow.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@50919 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'ext/date')
-rw-r--r-- | ext/date/date_parse.c | 22 |
1 files changed, 16 insertions, 6 deletions
diff --git a/ext/date/date_parse.c b/ext/date/date_parse.c index 239e2643b0..68ae7a675c 100644 --- a/ext/date/date_parse.c +++ b/ext/date/date_parse.c @@ -68,6 +68,7 @@ static const char *abbr_months[] = { static void s3e(VALUE hash, VALUE y, VALUE m, VALUE d, int bc) { + VALUE vbuf = 0; VALUE c = Qnil; if (!RB_TYPE_P(m, T_STRING)) @@ -164,10 +165,11 @@ s3e(VALUE hash, VALUE y, VALUE m, VALUE d, int bc) { char *buf; - buf = ALLOCA_N(char, ep - bp + 1); + buf = ALLOCV_N(char, vbuf, ep - bp + 1); memcpy(buf, bp, ep - bp); buf[ep - bp] = '\0'; iy = cstr2num(buf); + ALLOCV_END(vbuf); } set_hash("year", iy); } @@ -189,10 +191,11 @@ s3e(VALUE hash, VALUE y, VALUE m, VALUE d, int bc) { char *buf; - buf = ALLOCA_N(char, ep - bp + 1); + buf = ALLOCV_N(char, vbuf, ep - bp + 1); memcpy(buf, bp, ep - bp); buf[ep - bp] = '\0'; im = cstr2num(buf); + ALLOCV_END(vbuf); } set_hash("mon", im); } @@ -211,10 +214,11 @@ s3e(VALUE hash, VALUE y, VALUE m, VALUE d, int bc) { char *buf; - buf = ALLOCA_N(char, ep - bp + 1); + buf = ALLOCV_N(char, vbuf, ep - bp + 1); memcpy(buf, bp, ep - bp); buf[ep - bp] = '\0'; id = cstr2num(buf); + ALLOCV_END(vbuf); } set_hash("mday", id); } @@ -420,6 +424,7 @@ VALUE date_zone_to_diff(VALUE str) { VALUE offset = Qnil; + VALUE vbuf = 0; long l, i; char *s, *dest, *d; @@ -428,7 +433,7 @@ date_zone_to_diff(VALUE str) l = RSTRING_LEN(str); s = RSTRING_PTR(str); - dest = d = ALLOCA_N(char, l + 1); + dest = d = ALLOCV_N(char, vbuf, l + 1); for (i = 0; i < l; i++) { if (isspace((unsigned char)s[i]) || s[i] == '\0') { @@ -543,9 +548,10 @@ date_zone_to_diff(VALUE str) goto num; } if (strpbrk(RSTRING_PTR(str), ",.")) { + VALUE astr = 0; char *a, *b; - a = ALLOCA_N(char, RSTRING_LEN(str) + 1); + a = ALLOCV_N(char, astr, RSTRING_LEN(str) + 1); strcpy(a, RSTRING_PTR(str)); b = strpbrk(a, ",."); *b = '\0'; @@ -557,6 +563,7 @@ date_zone_to_diff(VALUE str) f_expt(INT2FIX(10), LONG2NUM((long)strlen(b)))), INT2FIX(60)); + ALLOCV_END(astr); goto num; } { @@ -605,6 +612,7 @@ date_zone_to_diff(VALUE str) } RB_GC_GUARD(str); ok: + ALLOCV_END(vbuf); return offset; } @@ -1975,7 +1983,8 @@ parse_ddd_cb(VALUE m, VALUE hash) set_hash("zone", s5); if (*cs5 == '[') { - char *buf = ALLOCA_N(char, l5 + 1); + VALUE vbuf = 0; + char *buf = ALLOCV_N(char, vbuf, l5 + 1); char *s1, *s2, *s3; VALUE zone; @@ -1997,6 +2006,7 @@ parse_ddd_cb(VALUE m, VALUE hash) if (isdigit((unsigned char)*s1)) *--s1 = '+'; set_hash("offset", date_zone_to_diff(rb_str_new2(s1))); + ALLOCV_END(vbuf); } RB_GC_GUARD(s5); } |