rgw: check NULL before dereference

Another coverity fix.

Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>

diff --git a/src/rgw/rgw_acl_s3.cc b/src/rgw/rgw_acl_s3.cc
index 3680bc4cc1ff17531041f1c9ea0cd7c6f2d8b8fd..e1c81e73ac8d3ba9c0330e195d7a125cf48b983c 100644 (file)
--- a/src/rgw/rgw_acl_s3.cc
+++ b/src/rgw/rgw_acl_s3.cc
@@ -300,6 +300,9 @@ bool RGWAccessControlList_S3::create_canned(string id, string name, string canne
 bool RGWAccessControlPolicy_S3::xml_end(const char *el) {
   RGWAccessControlList_S3 *s3acl =
       (RGWAccessControlList_S3 *)find_first("AccessControlList");
+  if (!s3acl)
+    return false;
+
   acl = *s3acl;
 
   ACLOwner *owner_p = (ACLOwner_S3 *)find_first("Owner");