🐛 Add a null test for texture

Author: davidAlgis

Plugin/PluginInteropUnityCUDA/src/Texture/texture.cpp

@@ -4,6 +4,14 @@
 Texture::Texture(void *textureHandle, int textureWidth, int textureHeight,
                  int textureDepth)
 {
+    if (textureHandle == nullptr)
+    {
+        Log::log().debugLogError(
+            "The texture ptr is null, please create it in Unity and then send "
+            "it with GetNativePtr function.");
+        return;
+    }
+
     _textureHandle = textureHandle;
     _textureWidth = textureWidth;
     _textureHeight = textureHeight;
@@ -69,8 +77,8 @@ int Texture::mapTextureToSurfaceObject()
         CUDA_CHECK_RETURN(cudaGetLastError());
     }
     CUDA_CHECK_RETURN(cudaMemcpy(d_surfObjArray, _surfObjArray,
-                          _textureDepth * sizeof(cudaSurfaceObject_t),
-                          cudaMemcpyHostToDevice));
+                                 _textureDepth * sizeof(cudaSurfaceObject_t),
+                                 cudaMemcpyHostToDevice));
     return SUCCESS_INTEROP_CODE;
 }
 
@@ -134,9 +142,9 @@ cudaSurfaceObject_t Texture::getSurfaceObject(int indexInArray) const
     }
 
     // to use a single surface object in a kernel
-    // we can use directly the surface object that 
+    // we can use directly the surface object that
     // is on host side, because cudaSurfaceObject_t is a
     // typename for unsigned long long which can be directly
-    // send to kernel as it's managed memory ? 
+    // send to kernel as it's managed memory ?
     return _surfObjArray[indexInArray];
 }

Plugin/PluginInteropUnityCUDA/src/Texture/texture_D3D11.cpp

@@ -22,6 +22,13 @@ Texture_D3D11::~Texture_D3D11()
 
 int Texture_D3D11::registerTextureInCUDA()
 {
+    if (_textureHandle == nullptr)
+    {
+        Log::log().debugLogError(
+            "The texture ptr is null, please create it in Unity and then send "
+            "it with GetNativePtr function.");
+        return -1;
+    }
     // texture2D and texture2D array are ID3D11Texture2D in Unity for DX11
     _texUnityDX11 = (ID3D11Texture2D *)_textureHandle;
 
@@ -39,23 +46,21 @@ int Texture_D3D11::registerTextureInCUDA()
         format == DXGI_FORMAT_R32G32_TYPELESS ||
         format == DXGI_FORMAT_R8_TYPELESS)
     {
-        Log::log().debugLogError(
-            "Texture of type " + std::to_string(format) +
-            " cannot be registered in CUDA." +
-            " It may comes from the fact that you can\'t used RenderTexture for "
-            "DX11 but only Texture2D.");
+        Log::log().debugLogError("Texture of type " + std::to_string(format) +
+                                 " cannot be registered in CUDA." +
+                                 " It may comes from the fact that you can\'t "
+                                 "used RenderTexture for "
+                                 "DX11 but only Texture2D.");
         return -1;
     }
 
-
     // register the texture to cuda : it initialize the _pGraphicsResource
     CUDA_CHECK_RETURN(cudaGraphicsD3D11RegisterResource(
         &_graphicsResource, _texUnityDX11, cudaGraphicsRegisterFlagsNone));
 
     return SUCCESS_INTEROP_CODE;
 }
 
-
 int Texture_D3D11::unregisterTextureInCUDA()
 {
     CUDA_CHECK_RETURN(cudaGraphicsUnregisterResource(_graphicsResource));
@@ -64,13 +69,14 @@ int Texture_D3D11::unregisterTextureInCUDA()
 
 int Texture_D3D11::generateMips()
 {
-    // we generate the shader resource in case the user want to use them (eg. for mips generation)
-    GRUMBLE(_renderAPI->createShaderResource(_texUnityDX11, &_shaderResources), "Could not create shader resources. Cancel mips genereation");
+    // we generate the shader resource in case the user want to use them (eg.
+    // for mips generation)
+    GRUMBLE(_renderAPI->createShaderResource(_texUnityDX11, &_shaderResources),
+            "Could not create shader resources. Cancel mips genereation");
     _renderAPI->getCurrentContext()->GenerateMips(_shaderResources);
     return SUCCESS_INTEROP_CODE;
 }
 
-
 int Texture_D3D11::copyUnityTextureToAPITexture()
 {
     _renderAPI->copyTextures2D(_texBufferInterop, _texUnityDX11);

Plugin/PluginInteropUnityCUDA/src/Texture/texture_OpenGLCoreES.cpp

@@ -18,6 +18,13 @@ Texture_OpenGLCoreES::~Texture_OpenGLCoreES()
 
 int Texture_OpenGLCoreES::registerTextureInCUDA()
 {
+    if (_textureHandle == nullptr)
+    {
+        Log::log().debugLogError(
+            "The texture ptr is null, please create it in Unity and then send "
+            "it with GetNativePtr function.");
+        return -1;
+    }
     // if depth is < 2 it's a texture2D, else it's a texture2DArray
     _texTarget = _textureDepth < 2 ? GL_TEXTURE_2D : GL_TEXTURE_2D_ARRAY;