Fix memory corruption in Ogre2GpuRays (Garden) (#788)

* Fix memory corruption in Ogre2GpuRays

Order of destruction was inverted

Signed-off-by: Matias N. Goldberg <[email protected]>

Author: darksylinc

ogre2/src/Ogre2GpuRays.cc

@@ -603,56 +603,29 @@ void Ogre2GpuRays::Destroy()
   auto engine = Ogre2RenderEngine::Instance();
   auto ogreRoot = engine->OgreRoot();
   auto textureGpuManager = ogreRoot->getRenderSystem()->getTextureGpuManager();
-  if (this->dataPtr->cubeUVTexture)
-  {
-    textureGpuManager->destroyTexture(this->dataPtr->cubeUVTexture);
-    this->dataPtr->cubeUVTexture = nullptr;
-  }
-  if (this->dataPtr->colorTexture)
-  {
-    textureGpuManager->destroyTexture(this->dataPtr->colorTexture);
-    this->dataPtr->colorTexture = nullptr;
-  }
-  if (this->dataPtr->depthTexture)
-  {
-    textureGpuManager->destroyTexture(this->dataPtr->depthTexture);
-    this->dataPtr->depthTexture = nullptr;
-  }
-  if (this->dataPtr->particleTexture)
-  {
-    textureGpuManager->destroyTexture(this->dataPtr->particleTexture);
-    this->dataPtr->particleTexture = nullptr;
-  }
-  if (this->dataPtr->particleDepthTexture)
-  {
-    textureGpuManager->destroyTexture(this->dataPtr->particleDepthTexture);
-    this->dataPtr->particleDepthTexture = nullptr;
-  }
 
   Ogre::CompositorManager2 *ogreCompMgr = ogreRoot->getCompositorManager2();
 
   // remove 1st pass textures, material, compositors
   for (auto i : this->dataPtr->cubeFaceIdx)
   {
-    if (this->dataPtr->firstPassTextures[i])
-    {
-      ogreRoot->getRenderSystem()->getTextureGpuManager()->destroyTexture(
-         this->dataPtr->firstPassTextures[i]);
-      this->dataPtr->firstPassTextures[i] = nullptr;
-    }
     if (this->dataPtr->ogreCompositorWorkspace1st[i])
     {
       ogreCompMgr->removeWorkspace(
           this->dataPtr->ogreCompositorWorkspace1st[i]);
       this->dataPtr->ogreCompositorWorkspace1st[i] = nullptr;
     }
+    if (this->dataPtr->firstPassTextures[i])
+    {
+      textureGpuManager->destroyTexture(this->dataPtr->firstPassTextures[i]);
+      this->dataPtr->firstPassTextures[i] = nullptr;
+    }
   }
 
   // remove 2nd pass texture, material, compositor
   if (this->dataPtr->secondPassTexture)
   {
-    ogreRoot->getRenderSystem()->getTextureGpuManager()->destroyTexture(
-       this->dataPtr->secondPassTexture);
+    textureGpuManager->destroyTexture(this->dataPtr->secondPassTexture);
     this->dataPtr->secondPassTexture = nullptr;
   }
 
@@ -662,6 +635,32 @@ void Ogre2GpuRays::Destroy()
     this->dataPtr->ogreCompositorWorkspace2nd = nullptr;
   }
 
+  if (this->dataPtr->cubeUVTexture)
+  {
+    textureGpuManager->destroyTexture(this->dataPtr->cubeUVTexture);
+    this->dataPtr->cubeUVTexture = nullptr;
+  }
+  if (this->dataPtr->colorTexture)
+  {
+    textureGpuManager->destroyTexture(this->dataPtr->colorTexture);
+    this->dataPtr->colorTexture = nullptr;
+  }
+  if (this->dataPtr->depthTexture)
+  {
+    textureGpuManager->destroyTexture(this->dataPtr->depthTexture);
+    this->dataPtr->depthTexture = nullptr;
+  }
+  if (this->dataPtr->particleTexture)
+  {
+    textureGpuManager->destroyTexture(this->dataPtr->particleTexture);
+    this->dataPtr->particleTexture = nullptr;
+  }
+  if (this->dataPtr->particleDepthTexture)
+  {
+    textureGpuManager->destroyTexture(this->dataPtr->particleDepthTexture);
+    this->dataPtr->particleDepthTexture = nullptr;
+  }
+
   if (this->scene)
   {
     Ogre::SceneManager *ogreSceneManager = this->scene->OgreSceneManager();