什么时候在 Excel XLL 中释放 FP * 是安全的？

Question

MSDN 声明：

...因此，当返回 DLL 创建的字符串或浮点数时 数组，您有以下选择：

• 设置一个指向动态分配缓冲区的持久指针，返回该指针。在下次调用函数 (1) 时，检查 指针不为空，（2）释放之前分配的资源 调用并将指针重置为空，（3）将指针重新用于新的 分配的内存块。 ...

当我调用 free 时，出现以下错误对话框：

MSVC++ 调试库检测到堆损坏：正常后 block(#135) at 0x....... CRT 检测到应用程序写入 修复缓冲区结束后的内存。

这是我的代码：

FP * g_FP;

extern "C" FP * __stdcall xllFill(long rows, long cols) {

    if (g_FP != NULL) {
        free(g_FP);
        g_FP = NULL;
    }
    g_FP = (FP *)malloc(rows * cols * sizeof(double) + 2 * sizeof(unsigned short int));

    for (int i = 0; i < rows * cols; i++) {
        (*g_FP).data[i] = (double)i;
    }
    (*g_FP).rows = (unsigned short int)rows;
    (*g_FP).cols = (unsigned short int)cols;
    return g_FP;
}

我对 C++ 有点生疏，但我无法弄清楚为什么这不起作用。

Answer 1

FP 的声明如下：

typedef struct _FP
{
    unsigned short int rows;
    unsigned short int columns;
    double array[1];        /* Actually, array[rows][columns] */
} FP;

您假设 FP 已打包并且不包含任何填充。我不知道 XLL 是如何编译的，但我认为很可能在 columns 和 array 之间有填充来排列该 array 是 8 字节对齐的。使用默认设置，MSVC 对于 sizeof(FP) 返回 16，这支持了我的假设。

将您的分配更改为：

g_FP = malloc((rows*cols-1)*sizeof(double) + sizeof(*g_FP));

即使这不是问题的原因，上面的分配也是逻辑上正确的形式。

否则我看不出你的代码有什么问题。我认为您可以更明确地将 g_FP 初始化为 NULL 但这是一个小问题。

Answer 2

sizeof(FP) 是 16，因为行和列最终对齐（大概）。我在手动尺寸计算中不允许这样做。

更好的代码是：

g_FP = (FP *)malloc(sizeof(sizeof(FP) - sizeof(double) + rows * cols * sizeof(double)); // -sizeof(double) to account for the double[1] already in the struct def.

Answer 3

为自己省去一些麻烦，请使用 http://nxll.codeplex.com 中的 FP 类。它的文档是 http://nxll.codeplex.com/wikipage?title=FP。