c++ malloc 分段错误

Question

我有一个关于 malloc() 的问题。这很奇怪。 我的代码如下。 我使用随机生成器来生成数组的元素。 该数组由 malloc() 打开。 如果数组大小小于8192，就可以。如果大小大于8192，则显示段错误。

void random_generator(int num, int * array) {

  srand((unsigned)time(0)); 
  int random_integer; 
  for(int index=0; index< num; index++){ 
    random_integer = (rand()%10000)+1; 
    *(array+index) = random_integer; 
    cout << index << endl;
  } 
}

int main() {
  int array_size = 10000;
  int *input_array;
  input_array = (int*) malloc((array_size));
  random_generator(8192, input_array);    // if the number is larger than 8192, segment fault
  free(input_array);
}

Answer 1

malloc() 采用字节大小，而不是元素数量。 int 的大小通常为 4 个字节，因此您实际上只为 2500 个整数分配了足够的内存。您正在分配 array_size 字节，而您应该分配 array_size * sizeof(int) 字节。

修复

因此，该错误将通过input_array = (int*) malloc(array_size * sizeof(int));

。 PS 永远不要假设您知道 int 或任何其他数据类型，因为它依赖于平台。始终使用sizeof()。

PPS 这实际上是一个 C 问题，而不是一个 C++ 问题。如果您实际上使用 C++，您应该考虑使用 new 和 delete [] 而不是 malloc() 和 free()< /code>，或者更好的是使用 std::vector 而不是数组，正如 Neil 指出的那样。

Answer 2

您想要：

input_array = (int*) malloc( array_size * sizeof(int) );

您可能会考虑更简单的：

input_array = new int[ array_size ];
// stuff
delete [] input_array;

甚至：

std::vector <int> input_array( array_size );

而不必担心调用 free 或 delete，或者担心异常。

Answer 3

看到它的 C++，你最好执行以下操作：

int main() 
{
  int array_size = 8192;
  int *input_array = new int[array_size];
  random_generator(array_size, input_array);
  delete[] input_array;
}

编辑：或者更好：

#include <vector>

int main() 
{
  int array_size = 8192;
  std::vector< int > array;
  array.resize( array_size );
  random_generator(array_size, &array.front());
}

甚至不用担心释放：D

Answer 4

input_array = (int*) malloc(sizeof(int) * (array_size));

这是因为 malloc 的参数是字节数，而 int 一般为 4 个字节长。

Answer 5

您使用的 malloc 调用是错误的。你应该使用：

malloc(sizeof(int) * array_size)

我认为。

Answer 6

您正在分配 array_size 字节，但需要 array_size 整数。尝试在 malloc 中分配 array_size*sizeof(int) 字节。

Answer 7

其他人都已经掩盖了动态分配中的错误。
但你真的想要动态分配吗？

int main() {
  int const array_size = 10000;
  int input_array[array_size];
  random_generator(8192, input_array); 
}