From: huanwen ren <ren.huanwen@zte.com.cn>
Date: Mon, 19 Sep 2016 00:50:47 +0000 (+0800)
Subject: mds/server: clean up handle_client_open()
X-Git-Tag: v11.0.1~78^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=c170616d78e2229a570e116ff0b570e6685fecd3;p=ceph-ci.git

mds/server: clean up handle_client_open()

Signed-off-by: huanwen ren <ren.huanwen@zte.com.cn>
---

diff --git a/src/mds/Server.cc b/src/mds/Server.cc
index 40d9fd8db3f..98c8070969e 100644
--- a/src/mds/Server.cc
+++ b/src/mds/Server.cc
@@ -2889,18 +2889,16 @@ void Server::_lookup_ino_2(MDRequestRef& mdr, int r)
 void Server::handle_client_open(MDRequestRef& mdr)
 {
   MClientRequest *req = mdr->client_request;
+  dout(7) << "open on " << req->get_filepath() << dendl;
 
   int flags = req->head.args.open.flags;
   int cmode = ceph_flags_to_mode(flags);
-
-  bool need_auth = !file_mode_is_readonly(cmode) || (flags & O_TRUNC);
-
-  dout(7) << "open on " << req->get_filepath() << dendl;
-
   if (cmode < 0) {
     respond_to_request(mdr, -EINVAL);
     return;
   }
+  
+  bool need_auth = !file_mode_is_readonly(cmode) || (flags & O_TRUNC);
 
   if ((cmode & CEPH_FILE_MODE_WR) && mdcache->is_readonly()) {
     dout(7) << "read-only FS" << dendl;