glue: add error checks after invoking JNIEnv::NewStringUTF()

Author: stephengold

src/main/native/glue/co/ConvexHullBuilder.cpp

@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -169,6 +169,8 @@ JNIEXPORT jint JNICALL Java_com_github_stephengold_joltjni_ConvexHullBuilder_ini
         pMessage = "";
     }
     jstring message = pEnv->NewStringUTF(pMessage);
+    JPH_ASSERT(NULL != message);
+    EXCEPTION_CHECK(pEnv)
     pEnv->SetObjectArrayElement(storeMessage, (jsize)0, message);
     return (jint) result;
 }

src/main/native/glue/d/Decomposer.cpp

@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2025 Stephen Gold
+Copyright (c) 2025-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -105,9 +105,11 @@ class Decomposer : public IVHACD::IUserCallback, public IVHACD::IUserLogger {
         jfloat arg3 = 100.0;
 
         jstring arg4 = pAttachEnv->NewStringUTF(stageName);
+        JPH_ASSERT(NULL != arg4);
         EXCEPTION_CHECK(pAttachEnv)
 
         jstring arg5 = pAttachEnv->NewStringUTF(operationName);
+        JPH_ASSERT(NULL != arg5);
         EXCEPTION_CHECK(pAttachEnv)
 
         pAttachEnv->CallVoidMethod(mJavaObject, mUpdateMethodId,

src/main/native/glue/glue.h

@@ -186,6 +186,8 @@ extern std::atomic<JPH::uint32> gDeleteCount;
     const String& message = pResult->GetError(); \
     const char* const str = message.c_str(); \
     const jstring result = pEnv->NewStringUTF(str); \
+    JPH_ASSERT(NULL != result); \
+    EXCEPTION_CHECK(pEnv) \
     return result; \
   } \
   JNIEXPORT jboolean JNICALL hasErrorName(JNIEnv *, jobject, jlong resultVa) { \

src/main/native/glue/j/Joint.cpp

@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -58,6 +58,8 @@ JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_Joint_getName
     const String name = pJoint->mName;
     const char * const pName = name.c_str();
     const jstring result = pEnv->NewStringUTF(pName);
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }
 
@@ -86,5 +88,7 @@ JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_Joint_getParentNam
     const String name = pJoint->mParentName;
     const char * const pName = name.c_str();
     const jstring result = pEnv->NewStringUTF(pName);
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }

src/main/native/glue/j/Jolt.cpp

@@ -97,6 +97,8 @@ JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_Jolt_buildType
 #else
     result = pEnv->NewStringUTF("Release");
 #endif
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }
 
@@ -203,6 +205,8 @@ JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_Jolt_getConfigurat
   (JNIEnv *pEnv, jclass) {
     const char * const pString = GetConfigurationString();
     const jstring result = pEnv->NewStringUTF(pString);
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }
 
@@ -554,6 +558,7 @@ static void JavaTrace(const char *inFormat, ...) {
     // Create a Java string:
     jstring javaString = pAttachEnv->NewStringUTF(buffer);
     JPH_ASSERT(javaString != NULL);
+    EXCEPTION_CHECK(pAttachEnv)
     // Print to the configured PrintStream:
     pAttachEnv->CallVoidMethod(gTraceStream, gPrintMethodId, javaString);
     EXCEPTION_CHECK(pAttachEnv)
@@ -871,6 +876,8 @@ JNIEXPORT void JNICALL Java_com_github_stephengold_joltjni_Jolt_unregisterTypes
 JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_Jolt_versionString
   (JNIEnv *pEnv, jclass) {
     const jstring result = pEnv->NewStringUTF(STRING(JOLT_JNI_VERSION_STRING));
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }
 

src/main/native/glue/ph/PhysicsMaterial.cpp

@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -64,6 +64,8 @@ JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_PhysicsMaterial_ge
             = reinterpret_cast<PhysicsMaterial *> (materialVa);
     const char * const pName = pMaterial->GetDebugName();
     const jstring result = pEnv->NewStringUTF(pName);
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }
 

src/main/native/glue/r/Rtti.cpp

@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2025 Stephen Gold
+Copyright (c) 2025-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -26,6 +26,7 @@ SOFTWARE.
 #include "Jolt/Jolt.h"
 #include "Jolt/Core/RTTI.h"
 #include "auto/com_github_stephengold_joltjni_Rtti.h"
+#include "glue/glue.h"
 
 using namespace JPH;
 
@@ -40,5 +41,7 @@ JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_Rtti_getName
     const String& message = pRtti->GetName();
     const char* const str = message.c_str();
     const jstring result = pEnv->NewStringUTF(str);
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }

src/main/native/glue/st/StateRecorder.cpp

@@ -1,5 +1,5 @@
 /*
-Copyright (c) 2024-2025 Stephen Gold
+Copyright (c) 2024-2026 Stephen Gold
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -154,6 +154,8 @@ JNIEXPORT jstring JNICALL Java_com_github_stephengold_joltjni_StateRecorder_read
     pRecorder->Read(cppString);
     const char * const pResult = cppString.c_str();
     const jstring result = pEnv->NewStringUTF(pResult);
+    JPH_ASSERT(NULL != result);
+    EXCEPTION_CHECK(pEnv)
     return result;
 }