释放矩阵时出现分段错误

时间:2013-10-22 19:03:13

标签: c

我正在使用code :: blocks。

代码在dealloc_mat中2-3次迭代后释放矩阵时发送seg错误。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>


int **_mat;
int _lines, _columns;


void alloc_mat();
void dealloc_mat();

int main(int argc, char *argv[])
{
    _lines = 31, _columns = 22;

    alloc_mat();
    dealloc_mat();

    return 0;
}

void alloc_mat()
{
    int i, row, col;
    _mat = malloc(sizeof(int *) * _lines);

    for(i = 0 ; i < _lines ; i++)
    {
        int *tmpMatrix = malloc(sizeof(int) * _columns);
        _mat[i] = &tmpMatrix[i];
    }

    for(row = 0 ; row < _lines ; row++)
    {
        for(col = 0 ; col < _columns ; col++)
        {
            _mat[row][col] = 0;
        }
    }
}

void dealloc_mat()
{
    int row;

    for(row = 0; row < _lines; row++)
    {
        free(_mat[row]);
    }

    free(_mat);
}

3 个答案:

答案 0 :(得分:1)

这是错误:

_mat[i] = &tmpMatrix[i];

应该是

_mat[i] = &tmpMatrix[0];

或更好

_mat[i] = tmpMatrix;

答案 1 :(得分:1)

问题是你没有正确分配它。这样:

for(i = 0 ; i < _lines ; i++)
    {
        int *tmpMatrix = malloc(sizeof(int) * _columns);
        _mat[i] = &tmpMatrix[i];
    }

应该是这样的:

for(i = 0 ; i < _lines ; i++)
    {
        _mat[i] = malloc(sizeof(int) * _columns);
    }

此外,_mat_lines_columns是C中的保留标识符,您不应使用它们。保留以普通(即_mat)或标记(即struct _mat)命名空间中具有文件范围的下划线开头的任何标识符。

答案 2 :(得分:0)

以下是一些用于为字符串分配内存的函数,实际上是字符串数组,您可以根据需要轻松修改它们:

char **strings; // created with global scope (before main())   
void allocMemory(int numStrings, int max)
{
    int i;
    strings = malloc(sizeof(char*)*(numStrings+1));
    for(i=0;i<numStrings; i++) 
      strings[i] = malloc(sizeof(char)*max + 1);  
}

void freeMemory(int numStrings)
{
    int i;
    for(i=0;i<numStrings; i++)
        if(strings[i]) free(strings[i]);
    free(strings);  
}

以下是如何修改(和使用)上述内容:(注意,它实际上只是识别sizeof(type)中的差异)
请注意: 使用malloc()不会初始化值。如果要保证每个元素的初始值(例如 0 ),则可以使用calloc()代替。

void allocMemoryInt(int rows, int cols);
void freeMemoryInt(int numStrings);
int **array;

int main(void)
{
    allocMemoryInt(10, 3);
    freeMemoryInt(10);
    return 0;   
}

void allocMemoryInt(int rows, int cols)
{
    int i;
    array = malloc(sizeof(int *)*(rows));  //create memory for row pointers
    for(i=0;i<rows; i++) 
      array[i] = malloc(sizeof(int)*cols + 1);  //create memory for (row * col) elements
}

void freeMemoryInt(int rows)
{
    int i;
    for(i=0;i<rows; i++)  
        if(array[i]) free(array[i]);//call free for each row 
    free(array);  //free pointer array(will clean up everything allocated)
}