diff options
-rw-r--r-- | ChangeLog | 10 | ||||
-rw-r--r-- | strftime.c | 4 |
2 files changed, 12 insertions, 2 deletions
@@ -1,3 +1,13 @@ +Thu Mar 24 22:38:18 2016 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * strftime.c (FMT, FMTV): remove recursive-assignments to get rid + of undefined behavior. [ruby-core:74532] [Bug #12213] + +Thu Mar 24 22:38:05 2016 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * strftime.c (FMT, FMTV): remove recursive-assignments to get rid + of undefined behavior. [ruby-core:74532] [Bug #12213] + Thu Mar 24 17:44:02 2016 Nobuyoshi Nakada <nobu@ruby-lang.org> * strftime.c (FMT_PADDING): extract format for padding. diff --git a/strftime.c b/strftime.c index f7f3714c33..2f1888d457 100644 --- a/strftime.c +++ b/strftime.c @@ -287,7 +287,7 @@ rb_strftime_with_timespec(VALUE ftime, const char *format, size_t format_len, (precision <= 0) ? (precision = (def_prec)) : (precision)) #define FMT(def_pad, def_prec, fmt, val) \ do { \ - precision = FMT_PRECISION(def_prec); \ + FMT_PRECISION(def_prec); \ len = s - start; \ NEEDS(precision); \ rb_str_set_len(ftime, len); \ @@ -324,7 +324,7 @@ rb_strftime_with_timespec(VALUE ftime, const char *format, size_t format_len, } \ else { \ const char *fmts = FMT_PADDING(fmt, def_pad); \ - precision = FMT_PRECISION(def_prec); \ + FMT_PRECISION(def_prec); \ tmp = format_value(fmts, tmp, precision); \ rb_str_append(ftime, tmp); \ RSTRING_GETMEM(ftime, s, len); \ |