diff --git a/ChangeLog b/ChangeLog
index c3de9f7f..76250c36 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2010-06-12  Tatsuhiro Tsujikawa  <t-tujikawa@users.sourceforge.net>
+
+	Made protected member variable private.
+	* src/XmlRpcMethod.h
+	* src/XmlRpcMethodImpl.cc
+
 2010-06-12  Tatsuhiro Tsujikawa  <t-tujikawa@users.sourceforge.net>
 
 	Made protected member variable private.
diff --git a/src/XmlRpcMethod.h b/src/XmlRpcMethod.h
index 7a41cfaa..045b2e41 100644
--- a/src/XmlRpcMethod.h
+++ b/src/XmlRpcMethod.h
@@ -63,11 +63,11 @@ struct XmlRpcResponse;
 // subclass. If you add new XmlRpcMethod subclass, don't forget to add
 // it to XmlRpcMethodFactory.
 class XmlRpcMethod {
-protected:
+private:
   SharedHandle<OptionParser> _optionParser;
 
   Logger* _logger;
-
+protected:
   // Subclass must implement this function to fulfil XmlRpcRequest
   // req.  The return value of this method is used as a return value
   // of XML-RPC request.
@@ -91,6 +91,11 @@ protected:
   void applyChangeableGlobalOption(Option* dest, Option* src) const;
 
   BDE createErrorResponse(const Exception& e);
+
+  const SharedHandle<OptionParser>& getOptionParser() const
+  {
+    return _optionParser;
+  }
 public:
   XmlRpcMethod();
 
diff --git a/src/XmlRpcMethodImpl.cc b/src/XmlRpcMethodImpl.cc
index 2dbffb1d..fa306a8b 100644
--- a/src/XmlRpcMethodImpl.cc
+++ b/src/XmlRpcMethodImpl.cc
@@ -1016,7 +1016,7 @@ BDE GetGlobalOptionXmlRpcMethod::process
   BDE result = BDE::dict();
   for(std::map<std::string, std::string>::const_iterator i =
         e->getOption()->begin(), eoi = e->getOption()->end(); i != eoi; ++i) {
-    SharedHandle<OptionHandler> h = _optionParser->findByName((*i).first);
+    SharedHandle<OptionHandler> h = getOptionParser()->findByName((*i).first);
     if(!h.isNull() && !h->isHidden()) {
       result[(*i).first] = (*i).second;
     }