Question

我正在尝试在简单程序中使用C malloc / free函数。

void* alloc_array( const int size )
{
    void* p;

    p = malloc( size );
    if( p )
    {
        //memset(p, 0, size);
        return( p );
    }
    else
        return NULL;
}

void free_array( void* p )
{
    if( p )
    {
        free( p );
        p = NULL;
    }       
}

void** alloc_array_ptr( const int nPointers )
{
    void** p;   
    unsigned int size = AMOUNT_TO_ALLOC(nPointers, void*);

    p = malloc( size );
    if( p )
    {
        return( p );
    }
    else
        return( NULL );
}

void free_array_ptr( void** p, const int nPointers )
{
    unsigned int i;

    if( p )
    {
        for( i = 0; i < nPointers; i++ )
            if( p[i] )
            {
                free( p[i] );
                p[i] = NULL;
            }

        free(p);
        p = NULL;
    }
}
int main(int argc, char* agv[]) {
    // builds the string with the location of the ini file
    int inifile_size = 0;
    char* inifile;
    // gets the buffer size nedeed for the buffer
    inifile_size = GetCurrentDirectory(0, NULL);
    // allocates memory for the buffer
    inifile = (char*) alloc_array( AMOUNT_TO_ALLOC(inifile_size, char) );
    // fills the buffer with the current directory
    if( GetCurrentDirectory( inifile_size, inifile ) == 0 )
    {
        printf("GetCurrentDirectory failed! (ErrorCode: %u)\n", GetLastError());
        return( !MY_ERROR_OK );
    }
    strcat(inifile, "\\");
    strcat(inifile, "test.ini");    
    printf("INI File: %s\n", inifile);

    char** sessions;
    int nSessions = 0;
    int i;

    nSessions = getNumberOfSubkeys(str);
    printf("\nOK (%d sessions found)\n", nSessions);

    // allocating memory for the sessions array of char*
    sessions = (char**) alloc_array_ptr( nSessions );

    if( sessions == NULL )
    {
        printf("ERROR_IN_MEM_ALLOC (SESSIONS)\n");      
        return( MY_ERROR_MEM_ALLOC );
    }
    else
        printf("sessions: 0x%p\n", sessions);

    // allocating memory for each one of the char* in sessions
    for( i = 0; i < nSessions; i++ )
    {
        sessions[i] = (char*) alloc_array( AMOUNT_TO_ALLOC(MAX_KEY_LENGTH, char) );

        printf("sessions[%d]: 0x%p\n", i, sessions[i]);

        if( sessions[i] == NULL )
        {
            printf("ERROR_IN_MEM_ALLOC (SESSIONS[%d])\n", i);
            return( MY_ERROR_MEM_ALLOC );                   
        }
    }

    printf("\nSessions found:\n");

    if( readSubkeys( str, sessions ) == MY_ERROR_OK )
    {
        for( i = 0; i < nSessions; i++ )
        {
            printf("\t%s: ", sessions[i]);

            if( convert2ini(inifile, sessions[i]) == MY_ERROR_OK )
                printf("OK\n");
            else
            {
                printf("ERROR\n");
                return( !MY_ERROR_OK );
            }           
        }
    }
    else
    {
        printf("ERROR\n");
        return( MY_ERROR_OPEN_REGISTRY );   
    }

    // free the inifile array
    free_array(inifile);

    // free the array of char*
    free_array_ptr( (void**)sessions, nSessions );      

    // returns to OS
    return( MY_ERROR_OK ); 
}

这是一个将Windows注册表中的某些程序设置转换为ini文件（Windows7,64位，4Gb RAM）的小工具。

readSubkeys填充char *的预分配会话数组，其中包含从str中的注册表项获取的会话的名称

getNumberOfSubkeys返回注册表中给定的子键数

其余代码与此无关。问题是当我为每个char *分配内存时，在for循环中，malloc失败了。我正在使用CodeLite来编写这个程序，奇怪的是，当我逐步调试代码时，malloc不会失败。

这是程序的运行图像在这里。 running

有人可以，请给我一些建议吗？

Answer 1

奇怪的是，当我逐步调试代码时，malloc不会失败。

此语句可能意味着您的代码中的其他位置存在内存分配错误导致此块（例如malloc（）调用或断言）失败。如果没有更多的代码，我们真的没有足够的信息来完全调试它。

Answer 2

很抱歉，我必须将此作为答案发布。我仍然没有发表评论的权限。在第一个malloc语句中，您将malloc的返回类型转换为指向指针的指针。所以，在for循环中，当你说session [0]时，它将包含第一个位置0x00701760的地址（在第一个malloc之后）。您将for循环增加到会话[1]。这在技术上是不可能的。由于会话中没有下一个值。实际上你需要增加sessions [0]中包含的值。它可能像* sessions ++。我想说的是你应该将0x00701760增加到0x00701762并为此做一个malloc。所以0x00701762现在包含新分配的内存的地址。接下来是0x00701764然后是0x00701766。增加会话[]将导致错误或至少访问无效内存。希望这个建议有所帮助。

由于阿迪亚

Answer 3

谢谢大家的提示。

问题是我没有在程序早期正确分配内存。奇怪的是，这部分软件有效，但随后的内存分配失败了。我要解决的是我计算了我对malloc的内存的正确大小，包括添加分隔符和文件名的几个strcat调用。然后，该软件工作正常。我重写了有问题的部分，它看起来像这样。

    // builds the string with the current directory

    // gets the size nedeed for the buffer
    current_path_size = GetCurrentDirectory(0, NULL);

    // allocates memory for the buffer
    current_path = (char*) alloc_array( AMOUNT_TO_ALLOC(current_path_size, char) );

    // gets the current directory
    if( GetCurrentDirectory( current_path_size, current_path ) == 0 )
    {
        printf("GetCurrentDirectory failed! (ErrorCode: %u)\n", GetLastError());
        return( !MY_ERROR_OK );
    }

    // builds the string with the location of the ini file

    // calculates the total amount of memory to alloc
    total_size = current_path_size + strlen(CHAR_SEPARATOR) + strlen(PUTTY_FILENAME) + 1;

    // allocates memory for the full path + filename of the ini file
    ini_file = (char*) alloc_array(AMOUNT_TO_ALLOC(total_size, char));

    // fills the buffer with the path + separator + filename + leading '\0'
    strcat(ini_file, current_path);
    strcat(ini_file, CHAR_SEPARATOR);
    strcat(ini_file, PUTTY_FILENAME);
    ini_file[total_size - 1] = '\0';

没有响应的一件事就是为什么调试器没有崩溃代码。我仍然需要对此进行进一步的研究。

C malloc在调试/发布时的奇怪行为

3 个答案: