分割断层

Question

这是我的代码：

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

void main(int arge, char *argv[])
{
    FILE *f1;
    char ch,*fn="~/lyrics/";
    strcat(fn,argv[1]);
    strcat(fn,".txt");
    if( (f1 = fopen(fn,"r"))==NULL )
    {
        printf("\nWrong filename\n%s not found",argv[1]);
        return;
    }
    while((ch=getw(f1))!=EOF)
    {
        printf("%c",ch);
    }
}

我使用对其进行gcc -g -o file file.c了编译，并且编译器未给出任何错误消息。但是当我运行它时，我收到错误消息：

Segmentation fault (core dumped)
Bad permissions for mapped region at address 0x8048659 at 0x402C36B: strcat 
(in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so) by 0x80484D6: main (lyrics.c:9)

谁能帮帮我吗？

Answer 1

您在fn中没有足够的空间。通过对它进行覆盖，您将覆盖其堆栈分配的末尾并进入堆栈..因此出现分段错误。

您可以尝试以下方法：

char fn[255];
strcpy( fn, "~/lyrics/" );
strcat( fn, argv[1] );
strcat( fn, ".txt" );

您只需要确保整个路径和文件名可以容纳255个字符即可。

或者，您可以执行以下操作：

char* fn = NULL;
int argvLen = strlen( argv[1] );
fn = malloc( 9 + argvLen + 4 + 1 ); // Add 1 for null terminator.
strcpy( fn, "~/lyrics/" );
strcat( fn, argv[1] );
strcat( fn, ".txt" );

并且您已经为该字符串分配了足够的空间。只是别忘了在完成后释放它！

Answer 2

char *fn = "~/lyrics/";

由于fn可能指向只读内存中的字符串，因此应将其声明fn为的指针const char。

const char *fn = "~/lyrics/";

然后您会看到有一些错误。这是一个更好的解决方案：

char fn[MAX_SIZE] = "~/lyrics/";

这MAX_SIZE是的大小"~/lyrics/"，最大长度argv[1]和的长度之和".txt"。