c# - 堆已损坏：调用非托管函数时

Question

我正在JNIDiskInfoDll.dll从 C# 托管函数调用一个非托管且非常简单的 C++ 函数（位于 中），如下所示：

C++：

#include "stdafx.h"
#include "AtaSmart.h"

#include <iostream>
#include <string.h>

extern "C" __declspec(dllexport) char* __cdecl getSerial(LPTSTR inCStrIn)
{
    return "abcdefg";
}

C＃：

using System;
using System.Runtime.InteropServices;

namespace HardInfoRetriever
{
    class DiskInfoRetreiver
    {

        [DllImport("C:\\Users\\User1\\Documents\\Visual Studio 2017\\Projects\\HardInfoRetriever\\Debug\\JNIDiskInfoDll.dll",
            EntryPoint = "getSerial", CallingConvention = CallingConvention.Cdecl,
            BestFitMapping = false, ThrowOnUnmappableChar = true, CharSet = CharSet.Ansi)]

        public static extern String getSerial([MarshalAs(UnmanagedType.LPTStr)]String _driveletter_);
        public static String getSerialNumber(String driveletter)
        {
            try
            {
                return getSerial(driveletter);
            }
            catch (Exception e)
            {
                throw e;
            }
        }
    }
}

我的问题是，在运行应用程序后，我收到两个连续的错误，说projectName.exe has triggered a breakpointand Unhandled exception at 0x77110E23 (ntdll.dll) in projectName.exe: 0xC0000374: A heap has been corrupted (parameters: 0x7712E930).。知道虽然我遇到了这些错误，但该函数仍在返回所需的输出。

请注意，getSerialC 函数具有LPTSTR inCStrIn参数，因为我在删除return "abcdefg";错误仍然存​​在的整个代码（仅保留）之前使用它。

我不知道这里可能是什么问题。我试图将Charsetin更改DllImport为Unidcode，但仍然遇到相同的错误。请问有什么帮助吗？

Answer 1

感谢@PaulMcKnezie 的评论：

不要返回指针。让它工作的可靠方法是调用者提供缓冲区，函数将字符串复制到缓冲区。

所以我使用了与这里提到的相同方法相同的概念

返回带有 BSTR * 参数的字符串。

最后，下面是我的代码的最终工作版本：

C++：

extern "C" __declspec(dllexport) HRESULT __cdecl getSerial(LPTSTR inCStrIn, BSTR* inCStrOut)
{
    *inCStrOut= SysAllocString(L"abcdefg"); 
    return S_OK;
}

C＃：

using System;
using System.Runtime.InteropServices;

namespace HardInfoRetriever
{
    class DiskInfoRetreiver
    {

        [DllImport("C:\\Users\\User1\\Documents\\Visual Studio 2017\\Projects\\HardInfoRetriever\\Debug\\JNIDiskInfoDll.dll",
            EntryPoint = "getSerial", CallingConvention = CallingConvention.Cdecl,
            BestFitMapping = false, ThrowOnUnmappableChar = true, CharSet = CharSet.Ansi)]
        //[return: MarshalAs(UnmanagedType.LPTStr)]
        public static extern int getSerial([MarshalAs(UnmanagedType.LPTStr)] string _driveletter_, [MarshalAs(UnmanagedType.BStr)] out string serial);
        public static String getSerialNumber(string letter)
        {
            try
            {
                string serial;
                int result =  getSerial(letter, out serial);
                if (result == 0)
                {
                    return serial;
                }
                return null;
            }
            catch (Exception e)
            {
                throw e;
            }
        }
    }
}