无法在第一个地址之外访问malloc的内存

问题描述 投票:2回答:4

读取文件时,将为放置文件内容的字符串动态分配内存。这是在函数内部完成的,字符串作为char **str传递。

使用gdb我发现在**(str+i) = fgetc(aFile);行产生了一个seg错误

这是$ gdb a.out core的输出以及一些变量的值:

Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x0000000000400bd3 in readFile (aFile=0x994010, str=0x7ffd8b1a9338) at src/morse.c:59
59      **(str + i) = fgetc(aFile);
(gdb) print i
$1 = 1
(gdb) print **(str + 0)
$2 = 65 'A'
(gdb) print *(str + 0)
$3 = 0x994250 "A"
(gdb) print (str + 0)
$4 = (char **) 0x7ffd8b1a9338
(gdb) print **(str + 1)
Cannot access memory at address 0x0
(gdb) print *(str + 1)
$5 = 0x0
(gdb) print (str + 1)
$6 = (char **) 0x7ffd8b1a9340

这是相关功能:

int readFile(FILE *aFile, char **str) // puts a file into a string
{
  int fileSize = 0;
  int i = 0;

  // count the length of the file string
  while(fgetc(aFile) != EOF)
    fileSize++;

  // malloc enough space for the string
  *str = (char *)malloc(sizeof(char) * (fileSize + 1 + 1)); // one for null, one for extra space
  if(!(*(str)))
    printf("ERROR: *str == NULL\n");

  // rewind() to the start of the file
  rewind(aFile);

  // put the file into a string
  for(i = 0; i < fileSize; i++)
    **(str + i) = fgetc(aFile);
  **(str + i - 1) = '\0';

  return 0;
}

为什么可以访问内存的开头(因为没有更好的术语)而不是更多? **级看似连续的内存和*级别的非连续内存有什么区别?

其余的代码可以在GitHub上找到here

c malloc dynamic-memory-allocation
4个回答
7
投票

它应该是*(*str+i)而不是**(str+i)。你已经为*str指针分配了内存。


1
投票

在您的代码中,指向已分配内存块的指针是*str,而不是**str。因此,您需要设置*((*str) + i)的值,或者等效地设置(*str)[i]的值。


0
投票

应该是*(* str + i),并且你应该被风格警察开除,因为它首先使用这种编码风格。

使用某种模式

char * myStr = * str =(char *)malloc ....

* myStr [i] =字节

....


0
投票

@B. Shankar已经给你一个答案,但我想我应该指出一些改进。

除非您需要使用指针算法,否则您的代码中绝对不需要所有循环,您可以这样做

int readFile(FILE *aFile, char **str) // puts a file into a string
{
  int fileSize;

  // count the length of the file string
  fseek(aFile, 0L, SEEK_END);
  fileSize = ftell(aFile);
  fseek(aFile, 0L, SEEK_SET);

  // malloc enough space for the string
  *str = malloc(fileSize + 1 + 1)); // one for null, one for extra space
  if (*str == NULL)
      return -1;    
  if (fread(*str, 1, fileSize, aFile) != fileSize)
      return -1;
  return 0;
}
© www.soinside.com 2019 - 2024. All rights reserved.