Fixed memory leaks reported by ASan

WL#8338
    Memory allocated for Unit-test was not being released.

    Global data kept by ProtoBuf was not being released if plugin was
    uninstalled before server shutdown.

Author: akojima

rapid/plugin/x/src/xpl_plugin.cc

@@ -52,6 +52,13 @@ char *xpl_func_ptr(Xpl_status_variable_get callback)
   return ptr_cast.ptr;
 }
 
+
+static void exit_hook()
+{
+  google::protobuf::ShutdownProtobufLibrary();
+}
+
+
 /*
   Start the plugin: start webservers
 
@@ -65,6 +72,13 @@ char *xpl_func_ptr(Xpl_status_variable_get callback)
  */
 int xpl_plugin_init(MYSQL_PLUGIN p)
 {
+  static bool atexit_installed = false;
+  if (!atexit_installed)
+  {
+    atexit_installed = true;
+    atexit(exit_hook);
+  }
+
   xpl::Plugin_system_variables::clean_callbacks();
 
   xpl_init_performance_schema();

rapid/plugin/x/src/xpl_replication_observer.cc

@@ -26,7 +26,7 @@ using namespace xpl;
 
 int xpl_before_server_shutdown(Server_state_param *param)
 {
-  google::protobuf::ShutdownProtobufLibrary();
+ // google::protobuf::ShutdownProtobufLibrary();
 
   return 0;
 }

rapid/unittest/gunit/xplugin/obuffer_t.cc

@@ -21,6 +21,7 @@
 #include <string>
 #include <iostream>
 
+#include <boost/shared_ptr.hpp>
 #include <gtest/gtest.h>
 
 #include "ngs/protocol/output_buffer.h"
@@ -32,12 +33,14 @@ namespace im
   const ngs::Pool_config default_pool_config = { 0, 0, BUFFER_PAGE_SIZE };
 
   namespace tests {
+    std::vector<boost::shared_ptr<Page> > page_del;
 
     static void add_pages(Output_buffer &ob, const size_t no_of_pages, const size_t page_size)
     {
       for (size_t i = 0; i < no_of_pages; i++)
       {
-        ob.push_back(new Page(page_size));
+        page_del.push_back(boost::shared_ptr<Page>(new Page(page_size)));
+        ob.push_back(page_del.back().get());
       }
     }
 

rapid/unittest/gunit/xplugin/row_builder_t.cc

@@ -16,6 +16,7 @@
 
 #include <gtest/gtest.h>
 #include <boost/scoped_ptr.hpp>
+#include <boost/shared_ptr.hpp>
 #include <mysql/service_my_snprintf.h>
 #include <limits>
 #include <string>
@@ -43,11 +44,14 @@ typedef ngs::Page_pool Page_pool;
 
 const ngs::Pool_config default_pool_config = { 0, 0, BUFFER_PAGE_SIZE };
 
+static std::vector<boost::shared_ptr<ngs::Page> > page_del;
+
 static void add_pages(Output_buffer *ob, const size_t no_of_pages, const size_t page_size)
 {
   for (size_t i = 0; i < no_of_pages; i++)
   {
-    ngs::Resource<ngs::Page> page(new ngs::Page(page_size));
+    page_del.push_back(boost::shared_ptr<ngs::Page>(new ngs::Page(page_size)));
+    ngs::Resource<ngs::Page> page(page_del.back().get());
     ob->push_back(page);
   }
 }