From 23473d1f9fc33151f0d5804e0cca824794a0bfe6 Mon Sep 17 00:00:00 2001 From: knu Date: Mon, 13 Feb 2012 14:02:26 +0000 Subject: * vm_method.c (rb_method_boundp): obj.respond_to?(:a_protected_method) should return false because calling a protected method may cause NoMethodError if called from outside the class inheritance tree. Kernel#respond_to? is mostly used to test if it is safe to call a method, so the false positive should be avoided. [ruby-dev:40461] [ruby-dev:41739] [ruby-dev:41837] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@34582 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- vm_method.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'vm_method.c') diff --git a/vm_method.c b/vm_method.c index 3811f48246..9ffae4ccae 100644 --- a/vm_method.c +++ b/vm_method.c @@ -547,8 +547,10 @@ rb_method_boundp(VALUE klass, ID id, int ex) rb_method_entry_t *me = rb_method_entry(klass, id); if (me != 0) { - if ((ex & ~NOEX_RESPONDS) && (me->flag & NOEX_PRIVATE)) { - return FALSE; + if ((ex & ~NOEX_RESPONDS) && + ((me->flag & NOEX_PRIVATE) || + ((ex & NOEX_RESPONDS) && (me->flag & NOEX_PROTECTED)))) { + return 0; } if (!me->def) return 0; if (me->def->type == VM_METHOD_TYPE_NOTIMPLEMENTED) { @@ -1263,11 +1265,11 @@ rb_respond_to(VALUE obj, ID id) /* * call-seq: - * obj.respond_to?(symbol, include_private=false) -> true or false + * obj.respond_to?(symbol, include_all=false) -> true or false * - * Returns +true+ if _obj_ responds to the given - * method. Private methods are included in the search only if the - * optional second parameter evaluates to +true+. + * Returns +true+ if _obj_ responds to the given method. Private and + * protected methods are included in the search only if the optional + * second parameter evaluates to +true+. * * If the method is not implemented, * as Process.fork on Windows, File.lchmod on GNU/Linux, etc., @@ -1300,7 +1302,7 @@ obj_respond_to(int argc, VALUE *argv, VALUE obj) /* * call-seq: - * obj.respond_to_missing?(symbol, include_private) -> true or false + * obj.respond_to_missing?(symbol, include_all) -> true or false * * Hook method to return whether the _obj_ can respond to _id_ method * or not. -- cgit v1.2.3