Ошибка сегментации при использовании strcpy?

Question

Я пытаюсь определить путь во время компиляции путем пропускания:

-DDCROOTDEF='"/path/to/stuff"' 

на линии компиляции. Затем я попытаюсь использовать его в коде:

char * ptr_path; 
strcpy(ptr_path, DCROOTDEF); 
strcat(ptr_path,"/MainCommons/CommonLib/fonts/Arial.ttf"); 
char *pftf=ptr_path; 
gdImageStringFT(pimg,brect,iclr,pftf,pts,ang,ixp,iyp, (char *)cbuf); 

Это дает мне ошибку сегментации. Однако, если я попытаюсь напечатать строку сначала:

char * ptr_path; 
strcpy(ptr_path, DCROOTDEF); 
strcat(ptr_path,"/MainCommons/CommonLib/fonts/Arial.ttf"); 
char *pftf=ptr_path; 
printf("%s\n",pftf); 
gdImageStringFT(pimg,brect,iclr,pftf,pts,ang,ixp,iyp, (char *)cbuf); 

Это работает просто отлично. Какую сложность в указателе символов я не вижу здесь?

Благодаря

Answer 1

char * ptr_path; 
strcpy(ptr_path, DCROOTDEF); 

Вы никогда не инициализируют ptr_path.

Это не работает во втором фрагменте кода, вам просто не повезло, и он появляется для работы. Вы по-прежнему используете неинициализированный указатель и пытаетесь написать, кто знает, где в памяти.

Нужно инициализировать ptr_path, чтобы указать на массив char, длина которого не менее strlen(DCROOTDEF) + 1. Вы также должны проверить длину DCROOTDEF перед копированием ее содержимого в массив, чтобы быть уверенным, что он не слишком длинный. Вы можете сделать это вручную, используя strlen, или вы можете использовать функцию копирования с проверкой длины, такую ​​как strlcpy.

Answer 2

Указатель ptr_path не инициализирован, чтобы указывать на записываемую память, поэтому разворачивается его с помощью strcpy().

Вам необходимо позвонить, например. malloc() получить пространство, первое:

char * ptr_path = malloc(PATH_MAX); 

Или что-то подобное.

Answer 3

В

char * ptr_path; 
strcpy(ptr_path, DCROOTDEF); 
strcat(ptr_path,"/MainCommons/CommonLib/fonts/Arial.ttf"); 

указатель не связан с юридически выделенным блоком памяти, так что ваша программа работает в неопределенное поведение. Сначала необходимо выделить буфер, например, используя malloc(). Убедитесь, что буфер достаточно велик, чтобы удерживать результирующую строку вместе с завершающим нулевым символом.