字符串数组的分段错误

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”的数组元素执行此操作；在这些情况下，您只需将字符串文字的地址复制到数组元素即可。没有为这些元素分配新的内存，因此您不必担心释放它们。

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 的结果。