rgw: fix potential memory leaks in RGWPutCORS_ObjStore_S3::get_params

We should free `data` before return or jump to `done_err` label.

Signed-off-by: Yan Jun <yan.jun8@zte.com.cn>
(cherry picked from commit 69c6cf551754e51debb4e645716a89e29517f4ee)

diff --git a/src/rgw/rgw_rest_s3.cc b/src/rgw/rgw_rest_s3.cc
index 27ae9e696ab5793254221966dafe12ea14f795ba..3d70a9e215a1c83be446e889f009715094d1df5d 100644 (file)
--- a/src/rgw/rgw_rest_s3.cc
+++ b/src/rgw/rgw_rest_s3.cc
@@ -2138,7 +2138,8 @@ int RGWPutCORS_ObjStore_S3::get_params()
   if (s->aws4_auth_needs_complete) {
     int ret_auth = do_aws4_auth_completion();
     if (ret_auth < 0) {
-      return ret_auth;
+      r = ret_auth;
+      goto done_err;
     }
   }