Native (C++) library crash on Android on use of callback to C# managed function. (SIGSEGV Code = 2)

Hello,

I have spent most of my day hitting my head against this problem, would love if someone could sanity check it for me. Pretty please!

I have a native C++ library, which is multi-platform, but for Android compiles to a .so. Within this library I have a Logger utility that stores a function pointer to a managed C# delegate function. The logger utility receives any logged messages from the library during the runtime, and passes them back to Unity using a delegate function, and then logged using the Debug.* functions in Unity / C#.

This setup has worked just fine for a number of months, but something has changed somewhere, either in Unity or my own code, that now makes it results in the attached logcat output. I have the same problem on an S8 (Running 7.1) and a Pixel (running 8.0) , which both have used earlier versions of the same libraries with no issues. Notably this works fine on both Windows and iOS.

Note this is on Unity 2017.3.0f2, I'm not able to easily step back and check the current codebase on an older version as I am working against ARCore Preview 2. I have tried both Mono and IL2CPP and both crash in the same place.

On the C# side the relevant code looks like this..

Native / managed interface:

Code (CSharp):

1.  
2. // callback functionality
3. public delegate void DebugCallback(string message, MsgType type);
4.  
5. // addition to double check problems were not because of garbage collection of the delegate instance
6. public static DebugCallback debugCallback;
7.  
8. internal static class ARLibInterop
9. {
10.     [DllImport(ARLibInterface.pluginName, CallingConvention = CallingConvention.Cdecl)]
11.     internal static extern void InitialiseARSession(ARSessionConfig sessionConfig, DebugCallback debugCallback);
12. }
13.  
14. //...
15.  
16. public void Initialise()
17. {
18.      debugCallback = new DebugCallback(DebugMethod);
19.      debugCallback("testing before call", MsgType.LOG);
20.      ARLibInterop.InitialiseARSession(config, debugCallback);
21. }
22.  
23. #if UNITY_IOS
24. [MonoPInvokeCallback(typeof(DebugCallback))]
25. #endif
26. public static void DebugMethod(string message, MsgType type)
27. {
28.     // do stuff
29. }

MsgType is a simple enum:

Code (CSharp):

1. [Serializable]
2.     public enum MsgType
3.     {
4.         NONE = 0,
5.         ERROR = 1,
6.         WARN = 2,
7.         LOG = 3
8.     }

On the C++ side

Code (C):

1.  
2. enum MessageType
3. {
4.     MSG_NONE = 0,
5.     MSG_ERROR = 1,
6.     MSG_WARN = 2,
7.     MSG_LOG = 3
8. };
9.  
10. typedef void(__stdcall * DebugCallback) (const char * str, MessageType type);
11.  
12. extern "C" void InitialiseARSession(ARSessionConfig sessionConfig, DebugCallback debugCallback)
13. {
14. #if PLATFORM == PLATFORM_ANDROID
15.     __android_log_print(ANDROID_LOG_VERBOSE, APPNAME, "callback received by native interface is %p", debugCallback);
16. #endif
17.     debugCallback("test", MSG_LOG);
18.     //ARLib::Initialise(sessionConfig, debugCallback);
19. }

Going slightly insane as the function is valid before the native call, but crashes native side. This utility has been functioning fine on all platforms for months, so cannot think what is causing this now. What am I missing?!

 

Thank you! I hadn't come across Marshal.GetFunctionPointerForDelegate, will work that in. All the examples I had seen used the above so I have just stuck with that.

I figured out the problem about ten minutes after posting, though only just had the chance to check it was definitely it as I wasn't at my laptop. Like you suggested I started stripping out arguments which made me twig on the way home while running through my GitHub changelog for the hundredth time today.

Native:

Code (C):

1.     struct ARSessionConfig
2.     {
3.         MessageType verbosityLevel;
4.         bool flip;
5.  
6.         friend std::ostream& operator<<(std::ostream& os, const ARSessionConfig& config);
7.     };

Managed:

Code (CSharp):

1.     [StructLayout(LayoutKind.Sequential), Serializable]
2.     public struct ARSessionConfig
3.     {
4.         public MsgType loggerVerbosity;
5.         //public bool flip;
6.     }

An entire day, caused by a stray bool, which presumably threw the offset when hooking up the parameters native side in Android, where Windows still dealt with it.