From e9ce3e744dca3911153ba97b89cba7e7e20b8a5d Mon Sep 17 00:00:00 2001 From: usa Date: Fri, 30 Jun 2017 10:56:33 +0000 Subject: merge revision(s) 58696: [Backport #11384] autoload: always wait on loading thread We cannot assume autoload_provided/rb_feature_provided returning TRUE means it is safe to proceed without waiting. Another thread may call rb_provide_feature before setting the constant (via autoload_const_set). So we must wait until autoload is completed by another thread. Note: this patch was tested with an explicit rb_thread_schedule in rb_provide_feature to make the race condition more apparent as suggested by : > --- a/load.c > +++ b/load.c > @@ -563,6 +563,7 @@ rb_provide_feature(VALUE feature) > rb_str_freeze(feature); > > rb_ary_push(features, rb_fstring(feature)); > +rb_thread_schedule(); > features_index_add(feature, INT2FIX(RARRAY_LEN(features)-1)); > reset_loaded_features_snapshot(); > } * variable.c (check_autoload_required): do not assume a provided feature means autoload is complete, always wait if autoload is being performed by another thread. [ruby-core:81105] [Bug #11384] Thanks to git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_2_3@59221 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- variable.c | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'variable.c') diff --git a/variable.c b/variable.c index 738494110c..6947dffa2e 100644 --- a/variable.c +++ b/variable.c @@ -2020,6 +2020,17 @@ check_autoload_required(VALUE mod, ID id, const char **loadingpath) if (!RSTRING_PTR(file) || !*RSTRING_PTR(file)) { rb_raise(rb_eArgError, "empty file name"); } + + /* + * if somebody else is autoloading, we MUST wait for them, since + * rb_provide_feature can provide a feature before autoload_const_set + * completes. We must wait until autoload_const_set finishes in + * the other thread. + */ + if (ele->state && ele->state->thread != rb_thread_current()) { + return load; + } + loading = RSTRING_PTR(file); safe = rb_safe_level(); rb_set_safe_level_force(0); -- cgit v1.2.3