matz	2008-04-22 22:42:28 +0900 (Tue, 22 Apr 2008)

  New Revision: 16155

  Modified files:
    trunk/ChangeLog
    trunk/lib/drb/drb.rb

  Log:
    * lib/drb/drb.rb (DRb::DRbServer::check_insecure_method): should
      check method names by symbols, not by strings.  a patch from
      Kazuhiro NISHIYAMA <zn at mbf.nifty.com> in [ruby-dev:34487].

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/ChangeLog?r1=16155&r2=16154&diff_format=u
  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi/trunk/lib/drb/drb.rb?r1=16155&r2=16154&diff_format=u

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 16154)
+++ ChangeLog	(revision 16155)
@@ -1,3 +1,9 @@
+Tue Apr 22 22:40:57 2008  Yukihiro Matsumoto  <matz@r...>
+
+	* lib/drb/drb.rb (DRb::DRbServer::check_insecure_method): should
+	  check method names by symbols, not by strings.  a patch from
+	  Kazuhiro NISHIYAMA <zn at mbf.nifty.com> in [ruby-dev:34487].
+
 Tue Apr 22 22:15:54 2008  Nobuyoshi Nakada  <nobu@r...>
 
 	* misc/ruby-style.el (ruby-style-{case,label}-indent): up list from
Index: lib/drb/drb.rb
===================================================================
--- lib/drb/drb.rb	(revision 16154)
+++ lib/drb/drb.rb	(revision 16155)
@@ -1476,10 +1476,10 @@
       raise(ArgumentError, "#{any_to_s(msg_id)} is not a symbol") unless Symbol == msg_id.class
       raise(SecurityError, "insecure method `#{msg_id}'") if insecure_method?(msg_id)
       
-      if obj.private_methods.include?(msg_id.to_s)
+      if obj.private_methods.include?(msg_id)
 	desc = any_to_s(obj)
         raise NoMethodError, "private method `#{msg_id}' called for #{desc}"
-      elsif obj.protected_methods.include?(msg_id.to_s)
+      elsif obj.protected_methods.include?(msg_id)
 	desc = any_to_s(obj)
         raise NoMethodError, "protected method `#{msg_id}' called for #{desc}"
       else

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/