字符串数组的分段错误

Question

我遇到了一些字符串数组问题，这些问题似乎侵犯了保留的内存空间。代码太大，这里就不贴了，所以我把重要的部分贴在下面：

int main (  ){

 int i = 0, j = 0, k = 0, count = 0, numLinhas = 0, l = 0;
 char string[100][100];
 char line [17];
 char str[4];
 char str1[5];
 char str2[4];
 char str3[4];

 FILE *p;
 p = fopen("text.txt", "r");
 while(fgets(line, sizeof line, p)!=NULL){
  printf("%s", line);
  strncpy(string[i], line, 17);
  i++;
  numLinhas++; 
  }  
  fclose(p);
  char *temp[numLinhas]; 

之后它进入一个循环，其中将文件中包含的语句的含义存储在string [i] 中。 for的开头和三个例子如下所示：

    for (i = 0; i<numLinhas; i++){
    sscanf( string[i], "%s %s %s %s" ,str1, str,str2, str3);
    if(str[0]=='0' && str[1] == '0' && str[2]!= 'd') {
    temp[i] = "NOP";
    count++;
    }
    if(str[0]=='0'&& str[1] == '6' && str[2]!= 'd') {
    sprintf(temp[i],"%s,%s" , "MVI B", str2);
    count = count+2;
    }

    if(str[0]=='0'&& str[1] == '7' && str[2]!= 'd') {
    temp[i] = "RLC";
    count++;
    }

错误是偶然的 - 它并不总是发生。它通常发生在调用 sprintf 时。 是啊！以下是我正在加载的 txt 文件作为示例：

0000 21a 11r 00r
0003 7Ea
0004 21a 12r 00r
0007 46a
0008 80a
0009 21a 13r 00r
000C 77a
000D 3Ea 01a
000F 3Da
0010 76a
0011 0Ad
0012 03d
0013 01d

刚看到一些新东西。这是我得到的编译窗口：

marcos@john:~/Desktop$ ./paraler
0000 21a 11r 00rValor de l: 16

Valor de l: 1
0003 7Ea
Valor de l: 9
0004 21a 12r 00rValor de l: 16

Valor de l: 1
0007 46a
Valor de l: 9
0008 80a
Valor de l: 9
0009 21a 13r 00rValor de l: 16

Valor de l: 1
000C 77a
Valor de l: 9
000D 3Ea 01a
Valor de l: 13
000F 3Da
Valor de l: 9
0010 76a
Valor de l: 9
0011 0Ad
Valor de l: 9
0012 03d
Valor de l: 9
0013 01d
Valor de l: 9
string:0000 21a 11r 00r
string:

string:0003 7Ea

string:0004 21a 12r 00r
string:

string:0007 46a

string:0008 80a

string:0009 21a 13r 00r
string:

string:000C 77a

string:000D 3Ea 01a

string:000F 3Da

string:0010 76a

string:0011 0Ad

string:0012 03d

string:0013 01d

Segmentation fault

奇怪的是，我得到了一些字符串数组的空格......它与错误有什么关系吗？

Answer 1

您是否在调用sprintf 之前为每个temp[i] 分配内存？如果没有，那就是你的问题。

if (!strncmp(str, "06", 2) && str[2] != 'd')
{
  temp[i] = malloc(5 + strlen(str2) + 2);  // Thanks, philippe
  if (temp[i])
    sprintf(temp[i], "%s,%s", "MVI B", str2);
}

尽管现在您必须跟踪temp 的哪些元素被分配了malloc，以便您以后可以释放它们。

编辑

在程序结束时，您可以循环遍历 temp 数组并根据上面字符串的前导部分检查每个元素的内容，如果它们匹配，则使用 free 取消分配该元素：

for (i = 0; i < numLinhas; i++)
{
  if (strcnmp(temp[i], "MVI B", strlen("MVI B")) == 0)
    free(temp[i]);
}

您不需要对分配了“NOP”或“RLC”的数组元素执行此操作；在这些情况下，您只需将字符串文字的 address 复制到数组元素。没有为这些元素分配新内存，因此您不必担心释放它们。

Answer 2

我立即看到了一些问题：

编辑第一点可能不适用于您的环境，但您应该牢记这一点

• 您正试图从一个整数创建char *temp[numLinHas]，其值将在运行时确定。这在 C99 中是允许的，或者可以通过编译器扩展提供，但在较旧的 C 标准中，数组大小必须在编译时知道。你的代码可能真的是这样做的：

  int numLinHas = 0;
  char *temp[numLinHas];
  

• 另一个问题是，当您执行sprintf 时，您试图复制到temp 中的某些内容，而没有为指针分配内存来存储字符串。

Answer 3

if(str[0]=='0'&& str[1] == '6' && str[2]!= 'd') {
    /* allocate memory to store instruction - to be freed later */  
    temp[i] = malloc(5 + strlen(str2) + 2); /* +2 for the comma */
    sprintf(temp[i],"%s,%s" , "MVI B", str2);
    count = count+2;
    }

遇到以06开头且不以d结尾的助记符时会出现问题； sprintf() 写入非分配区域，因为 temp[i] 未初始化。您应该分配一些空间来存储 sprintf 的结果。