Что не так с этой функцией конкатенации?

Question

char* StrCat(char* dest, char* source) 
{ 
    char* retVal = dest; 

    while(*dest) 
     dest++; 

    while(*dest++ = *source++) ; 

    return retVal; 
} 

int main() 
{ 
    char* a = "One"; 
    char* b = "Two"; 

    char* x = StrCat(a, b); 

    printf("%s\n", x); 

    return 0; 
} 

Программа вылетает в первый раз при копировании из источника в пункт назначения, второй цикл while. (Access violation)

Answer 1

Содержимое переменных a и b (символьные строки «Один» и «Два») хранятся в сегментах только для чтения, содержимое защищено, вы не можете перезаписывать свои данные.

Плюс, функция StrCat() имеет мало логики.

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

Другим решением было бы распределить память динамически, используя malloc, эта память должна быть освобождена, когда она вам больше не понадобится.

Переосмыслить, как вы могли бы написать такую ​​функцию для ее исправления, но сначала исправить эту проблему с памятью.