操作字符串时出现分段错误？

Question

这段代码总结了我遇到的问题。我想将文件从源复制到指定的目标，我可以更改它的名称它是一个集成在我正在尝试创建的应用程序中的函数管理文件

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

 void concatenate_string(char *original, char *add)
{
   while(*original!='\0')
     original++;

   while(*add!='\0')
     {
        *original = *add;
         add++;
         original++;
     }
     *original = '\0';
}

 int main(int argc,char *argv[])
  {

    char *nom;
    char *path;
    printf("entrer a name \n");
    scanf("%s",nom);
    printf("entrer a pathh \n");
    scanf("%s",pathh);
    char *dest=(char*)malloc(strlen(nomm)+46+1);
    strcat(dest,"/home/ridaamine/Desktop/app/application/Files/");
    strcat(dest,nom);
    char *comand=(char*)malloc(strlen(name)+8+strlen(path)+1);
    strcat(comand,"cp -via ");
    strcat(comand,path);
    strcat(comand," ");
    strcat(comand,name);
    system(comand);
  }

Answer 1

你没有初始化nom

scanf("%s",nom);

要么

nom = malloc(SOME_SIZE);

然后，假设SOME_SIZE == 100

scanf("%99s", nom);

或

char nom[SOME_SIZE];

然后，让我们说SOME_SIZE == 100

scanf("%99s", nom);

当然这同样适用于path。

第二种解决方案更好，因为它更快not that much，并且您在使用后不需要free(nom)。在极少数情况下可能需要第二种情况，因为字符串的大小太大（> 8M）会溢出堆栈。

正如Weather Vane指出strcat也有问题，你应该第一次使用strcpy

strcat(dest,"/home/ridaamine/Desktop/app/application/Files/");
strcat(dest,nom);

应该

strcpy(dest,"/home/ridaamine/Desktop/app/application/Files/");
strcat(dest,nom);

很明显，这一次同样适用于command。

你终于有了一个你没有计入的空间

malloc(strlen(dest) + 8 + strlen(path) + 1 + 1 /* space " " */)

提示：您不需要投射malloc，所以不要，它可能隐藏一个潜在的错误。在取消引用指针之前，请务必检查 malloc 是否返回 NULL。

你也应该在完成后免费调用，这是你自己的代码修复

 int main(int argc,char *argv[])
  {

    char nom[100];
    char path[100];
    char _path[] = "/home/ridaamine/Desktop/app/application/Files/";
    char cp[] = "cp -via ";
    char space[] = " ";

    printf("entrer a name \n");
    scanf("%99s", nom);
    printf("entrer a path \n");
    scanf("%99s", path);

    char *dest = malloc(strlen(nom) + strlen(_path) + 1);
    if (dest == NULL)
    {
        printf("no more memory left.\n");
        return -1;
    }       
    strcpy(dest, _path);
    strcat(dest, nom);

    char *comand = malloc(strlen(dest) + strlen(cp) + strlen(space) + strlen(path) + 1);
    if (command == NULL)
    {
        free(dest);
        printf("no more memory left.\n");
        return -1;
    }

    strcpy(comand, cp);

    strcat(comand, path);
    strcat(comand, space);
    strcat(comand, dest);

    free(dest);
    system(comand);
    free(command);

    return 0; // always return from main
  }

Answer 2

第一个strcat() 必须在每种情况下更改为strcpy()，因为字符串尚未初始化为空字符串。而且第一个字符串dest肯定会太短。

char *dest=(char*)malloc(strlen(nomm)+46+1);    // this is too short
strcpy(dest,"/home/ridaamine/Desktop/app/application/Files/");
strcat(dest,nom);

char *comand=(char*)malloc(strlen(name)+8+strlen(path)+1);
strcpy(comand,"cp -via ");
strcat(comand,path);
strcat(comand," ");