Feature #5089 » 0001-More-removal-of-inadvertent-symbol-creation.patch
| eval_error.c | ||
|---|---|---|
|
rb_class2name(klass));
|
||
|
}
|
||
|
void
|
||
|
rb_print_undef_str(VALUE klass, VALUE name)
|
||
|
{
|
||
|
rb_name_error_str(name, "undefined method `%s' for %s `%s'",
|
||
|
RSTRING_PTR(name),
|
||
|
(TYPE(klass) == T_MODULE) ? "module" : "class",
|
||
|
rb_class2name(klass));
|
||
|
}
|
||
|
static int
|
||
|
sysexit_status(VALUE err)
|
||
|
{
|
||
| eval_intern.h | ||
|---|---|---|
|
VALUE rb_f_eval(int argc, VALUE *argv, VALUE self);
|
||
|
VALUE rb_make_exception(int argc, VALUE *argv);
|
||
|
NORETURN(void rb_method_name_error(VALUE, VALUE));
|
||
|
NORETURN(void rb_fiber_start(void));
|
||
|
NORETURN(void rb_print_undef(VALUE, ID, int));
|
||
|
NORETURN(void rb_print_undef_str(VALUE, VALUE));
|
||
|
NORETURN(void rb_vm_localjump_error(const char *,VALUE, int));
|
||
|
NORETURN(void rb_vm_jump_tag_but_local_jump(int, VALUE));
|
||
|
NORETURN(void rb_raise_method_missing(rb_thread_t *th, int argc, VALUE *argv,
|
||
| load.c | ||
|---|---|---|
|
static VALUE
|
||
|
rb_mod_autoload_p(VALUE mod, VALUE sym)
|
||
|
{
|
||
|
return rb_autoload_p(mod, rb_to_id(sym));
|
||
|
ID id = rb_check_id(sym);
|
||
|
if(!id) {
|
||
|
return Qnil;
|
||
|
}
|
||
|
return rb_autoload_p(mod, id);
|
||
|
}
|
||
|
/*
|
||
| object.c | ||
|---|---|---|
|
static ID id_eq, id_eql, id_match, id_inspect;
|
||
|
static ID id_init_copy, id_init_clone, id_init_dup;
|
||
|
static ID id_const_missing;
|
||
|
/*
|
||
|
* call-seq:
|
||
| ... | ... | |
|
else {
|
||
|
rb_scan_args(argc, argv, "11", &name, &recur);
|
||
|
}
|
||
|
id = rb_to_id(name);
|
||
|
id = rb_check_id(name);
|
||
|
if (!id) {
|
||
|
if(rb_is_const_name(name)) {
|
||
|
if (rb_method_basic_definition_p(CLASS_OF(mod), id_const_missing)) {
|
||
|
if (mod && rb_class_real(mod) != rb_cObject) {
|
||
|
rb_name_error_str(name, "uninitialized constant %s::%s",
|
||
|
rb_class2name(mod),
|
||
|
RSTRING_PTR(name));
|
||
|
} else {
|
||
|
rb_name_error_str(name, "uninitialized constant %s", RSTRING_PTR(name));
|
||
|
}
|
||
|
} else {
|
||
|
id = rb_to_id(name);
|
||
|
}
|
||
|
} else {
|
||
|
rb_name_error_str(name, "wrong constant name %s", RSTRING_PTR(name));
|
||
|
}
|
||
|
}
|
||
|
if (!rb_is_const_id(id)) {
|
||
|
rb_name_error(id, "wrong constant name %s", rb_id2name(id));
|
||
|
}
|
||
| ... | ... | |
|
id_init_copy = rb_intern("initialize_copy");
|
||
|
id_init_clone = rb_intern("initialize_clone");
|
||
|
id_init_dup = rb_intern("initialize_dup");
|
||
|
id_const_missing = rb_intern("const_missing");
|
||
|
for (i=0; conv_method_names[i].method; i++) {
|
||
|
conv_method_names[i].id = rb_intern(conv_method_names[i].method);
|
||
| proc.c | ||
|---|---|---|
|
return data->me.klass;
|
||
|
}
|
||
|
static void
|
||
|
void
|
||
|
rb_method_name_error(VALUE klass, VALUE str)
|
||
|
{
|
||
|
const char *s0 = " class";
|
||
| test/ruby/test_module.rb | ||
|---|---|---|
|
assert_raise(NameError) { c2::Bar }
|
||
|
assert_raise(NameError) { c2.const_get(:Bar) }
|
||
|
assert_raise(NameError) { c2.const_get(:Bar, false) }
|
||
|
assert_raise(NameError) { c2.const_get("Bar", false) }
|
||
|
assert_raise(NameError) { c2.const_get("BaR11", false) }
|
||
|
assert_raise(NameError) { Object.const_get("BaR11", false) }
|
||
|
c1.instance_eval do
|
||
|
def const_missing(x)
|
||
| ... | ... | |
|
assert_equal(:Bar, c2::Bar)
|
||
|
assert_equal(:Bar, c2.const_get(:Bar))
|
||
|
assert_equal(:Bar, c2.const_get(:Bar, false))
|
||
|
assert_equal(:Bar, c2.const_get("Bar"))
|
||
|
assert_equal(:Bar, c2.const_get("Bar", false))
|
||
|
v = c2.const_get("Bar11", false)
|
||
|
assert_equal("Bar11".to_sym, v)
|
||
|
assert_raise(NameError) { c1.const_get(:foo) }
|
||
|
end
|
||
| test/ruby/test_symbol.rb | ||
|---|---|---|
|
assert !Symbol.all_symbols.any? {|sym| sym.to_s == str}, msg
|
||
|
end
|
||
|
end
|
||
|
def test_no_inadvertent_symbol_creation3
|
||
|
feature50XX = '[ruby-core:384XX]'
|
||
|
c = Class.new do
|
||
|
def self.alias_method(str)
|
||
|
super(:puts, str)
|
||
|
end
|
||
|
end
|
||
|
s = "gadzoooks"
|
||
|
{:alias_method => ["#{s}1", NameError],
|
||
|
:autoload? => ["#{s}2", nil],
|
||
|
:const_get => ["A#{s}3", NameError],
|
||
|
:private_class_method => ["#{s}4", NameError],
|
||
|
:private_constant => ["#{s}5", NameError],
|
||
|
:private => ["#{s}6", NameError],
|
||
|
:protected => ["#{s}7", NameError],
|
||
|
:public => ["#{s}8", NameError],
|
||
|
:public_class_method => ["#{s}9", NameError],
|
||
|
:public_constant => ["#{s}10", NameError],
|
||
|
:remove_method => ["#{s}11", NameError],
|
||
|
:undef_method => ["#{s}12", NameError],
|
||
|
:untrace_var => ["#{s}13", NameError],
|
||
|
}.each do |meth, arr|
|
||
|
str, ret = arr
|
||
|
msg = "#{meth}(#{str}) #{feature50XX}"
|
||
|
if ret.is_a?(Class) && (ret < Exception)
|
||
|
assert_raises(ret){c.send(meth, str)}
|
||
|
else
|
||
|
assert(c.send(meth, str) == ret, msg)
|
||
|
end
|
||
|
assert !Symbol.all_symbols.any? {|sym| sym.to_s == str}, msg
|
||
|
end
|
||
|
end
|
||
|
end
|
||
| variable.c | ||
|---|---|---|
|
rb_secure(4);
|
||
|
rb_scan_args(argc, argv, "11", &var, &cmd);
|
||
|
id = rb_to_id(var);
|
||
|
id = rb_check_id(var);
|
||
|
if (!id) {
|
||
|
rb_name_error_str(var, "undefined global variable %s", RSTRING_PTR(var));
|
||
|
}
|
||
|
if (!st_lookup(rb_global_tbl, (st_data_t)id, &data)) {
|
||
|
rb_name_error(id, "undefined global variable %s", rb_id2name(id));
|
||
|
}
|
||
| ... | ... | |
|
int i;
|
||
|
st_data_t v;
|
||
|
ID id;
|
||
|
VALUE val;
|
||
|
if (rb_safe_level() >= 4 && !OBJ_UNTRUSTED(mod)) {
|
||
|
rb_raise(rb_eSecurityError,
|
||
| ... | ... | |
|
}
|
||
|
for (i = 0; i < argc; i++) {
|
||
|
id = rb_to_id(argv[i]);
|
||
|
val = argv[i];
|
||
|
id = rb_check_id(val);
|
||
|
if (!id) {
|
||
|
rb_name_error_str(val, "constant %s::%s not defined", rb_class2name(mod), RSTRING_PTR(val));
|
||
|
}
|
||
|
if (RCLASS_CONST_TBL(mod) && st_lookup(RCLASS_CONST_TBL(mod), (st_data_t)id, &v)) {
|
||
|
((rb_const_entry_t*)v)->flag = flag;
|
||
|
return;
|
||
| vm_method.c | ||
|---|---|---|
|
rb_mod_remove_method(int argc, VALUE *argv, VALUE mod)
|
||
|
{
|
||
|
int i;
|
||
|
ID id;
|
||
|
VALUE v;
|
||
|
for (i = 0; i < argc; i++) {
|
||
|
remove_method(mod, rb_to_id(argv[i]));
|
||
|
v = argv[i];
|
||
|
id = rb_check_id(v);
|
||
|
if (!id) {
|
||
|
rb_name_error_str(v, "method `%s' not defined in %s",
|
||
|
RSTRING_PTR(v), rb_class2name(mod));
|
||
|
}
|
||
|
remove_method(mod, id);
|
||
|
}
|
||
|
return mod;
|
||
|
}
|
||
| ... | ... | |
|
rb_mod_undef_method(int argc, VALUE *argv, VALUE mod)
|
||
|
{
|
||
|
int i;
|
||
|
ID id;
|
||
|
VALUE v;
|
||
|
for (i = 0; i < argc; i++) {
|
||
|
rb_undef(mod, rb_to_id(argv[i]));
|
||
|
v = argv[i];
|
||
|
id = rb_check_id(v);
|
||
|
if (!id) {
|
||
|
rb_method_name_error(mod, v);
|
||
|
}
|
||
|
rb_undef(mod, id);
|
||
|
}
|
||
|
return mod;
|
||
|
}
|
||
| ... | ... | |
|
static VALUE
|
||
|
rb_mod_alias_method(VALUE mod, VALUE newname, VALUE oldname)
|
||
|
{
|
||
|
ID oldid = rb_check_id(oldname);
|
||
|
if(!oldid) {
|
||
|
rb_print_undef_str(mod, oldname);
|
||
|
}
|
||
|
rb_alias(mod, rb_to_id(newname), rb_to_id(oldname));
|
||
|
return mod;
|
||
|
}
|
||
| ... | ... | |
|
set_method_visibility(VALUE self, int argc, VALUE *argv, rb_method_flag_t ex)
|
||
|
{
|
||
|
int i;
|
||
|
ID id;
|
||
|
VALUE v;
|
||
|
secure_visibility(self);
|
||
|
for (i = 0; i < argc; i++) {
|
||
|
rb_export_method(self, rb_to_id(argv[i]), ex);
|
||
|
v = argv[i];
|
||
|
id = rb_check_id(v);
|
||
|
if (!id) {
|
||
|
rb_print_undef_str(self, v);
|
||
|
}
|
||
|
rb_export_method(self, id, ex);
|
||
|
}
|
||
|
rb_clear_cache_by_class(self);
|
||
|
}
|
||