Что не так с этой программой?

Question

Я пытаюсь написать код конкатенации строк. Интересно, что в этом плохого? Можете ли вы помочь мне. Вот мой код.

#include <stdlib.h>

void strcat1(char *s, char *t)
{
    while(*s !='\0')
        s++;

    while((*s++=*t++)!= '\0')
    {

    }

}

int main()
{
char s[]= "hello";
char t[]= "world";
strcat1(s,t);
printf("%s", s);

return 0;
}

Я получаю этот вывод на codepad.org: Disallowed system call: SYS_socketcal Вот ссылка: http://codepad.org/Arz6U7YA

РЕДАКТИРОВАТЬ : Изменится ли char * s = "Hello" и char * t = "World" в основной функции как-то иначе?

Answer 1

s имеет место для 6 символов (а именно 'h', 'e', ​​'l', 'l', 'o' и '\ 0').

Вы пытаетесьнапишите на 5 символов больше, чем он может вместить.

Не делайте этого!

Попробуйте увеличить размер s до

int main()
{
char s[11] = "hello";
/* ... */

---

Редактироватьпосле редактирования OP

Изменение s в основном на

char *s = "hello";

изменяет s из массива с небольшим пространством на указатель на строковый литерал.

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

Answer 2

Что ж, большая проблема в том, что s недостаточно велика, чтобы сохранить результат;он имеет размер 6 символов (5 букв плюс 0), поэтому, как только вы начнете пытаться добавить содержимое t, вы переполните буфер.

Answer 3

Переполнение буфера. Вы не можете добавить в конец массива s, потому что он был выделен только с 6 символами (5 печатных и один \0).

Answer 4

Вы пытаетесь добавить постоянную строку. Этот сегмент памяти защищен. Создайте новый буфер, войдите туда, а затем верните его.