From: heister <heister@0785d39b-7218-0410-832d-ea1e28bc413d>
Date: Thu, 23 Jan 2014 17:26:42 +0000 (+0000)
Subject: fix invalid memory access caused by undefined order of construction of static members... 
X-Git-Url: https://gitweb.dealii.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=6d4c78288c59076f88f006379509472935d78d4d;p=dealii-svn.git

fix invalid memory access caused by undefined order of construction of static members (found with ASAN)

git-svn-id: https://svn.dealii.org/trunk@32289 0785d39b-7218-0410-832d-ea1e28bc413d
---

diff --git a/deal.II/include/deal.II/hp/mapping_collection.h b/deal.II/include/deal.II/hp/mapping_collection.h
index 512a6872b4..117d76bee0 100644
--- a/deal.II/include/deal.II/hp/mapping_collection.h
+++ b/deal.II/include/deal.II/hp/mapping_collection.h
@@ -148,16 +148,15 @@ namespace hp
   {
   private:
     /**
-     * A static MappingQ1
-     * object. We can't use the one
-     * in ::StaticMappingQ1 since
-     * we can't make sure that the
-     * constructor for that object
-     * is run before the
-     * constructor for the present
-     * static object.
+     * A static MappingQ1 object. We can't use the one in ::StaticMappingQ1
+     * since we can't make sure that the constructor for that object is run
+     * before we want to use the object (when constructing mapping_collection
+     * below).  Therefore we create a helper function which returns a
+     * reference to a static object that will be constructed the first time
+     * this function is called.
      */
-    static MappingQ1<dim,spacedim> mapping_q1;
+      static
+      MappingQ1<dim,spacedim>& return_static_mapping_q1();
 
   public:
     /**
diff --git a/deal.II/source/hp/mapping_collection.cc b/deal.II/source/hp/mapping_collection.cc
index 0d9fdcfa83..eb59d77463 100644
--- a/deal.II/source/hp/mapping_collection.cc
+++ b/deal.II/source/hp/mapping_collection.cc
@@ -82,16 +82,17 @@ namespace hp
 //---------------------------------------------------------------------------
 
 
-
   template<int dim, int spacedim>
-  MappingQ1<dim,spacedim>
-  StaticMappingQ1<dim,spacedim>::mapping_q1;
-
+  MappingQ1<dim,spacedim>& StaticMappingQ1<dim,spacedim>::return_static_mapping_q1()
+  {
+    static MappingQ1<dim,spacedim> mapping;
+    return mapping;
+  }
 
   template<int dim, int spacedim>
   MappingCollection<dim,spacedim>
   StaticMappingQ1<dim,spacedim>::mapping_collection
-    = MappingCollection<dim,spacedim>(mapping_q1);
+    = MappingCollection<dim,spacedim>(StaticMappingQ1<dim,spacedim>::return_static_mapping_q1());
 
 }