C - Ошибка с strcat

Question

Какая ошибка в коде? У меня есть ошибка

error C4996: 'strcat': This function or variable may be unsafe. Consider using strcat_s instead 

Что это значит? Другой вопрос - декларация структуры и прототипа функции является законной?

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

void main() 
{ 
    char *join(char *, char *); 
    printf("%s \n", join("duck", "soup")); 
} 

char *join(char *s1, char *s2) 
{ 
    struct { 
     char buf[256]; 
    } string; 
string.buf = "abcd";\\ new line. error l-value. 
    return strcat(strcpy(string.buf, s1), s2); 
} 

Новая линия - почему есть ошибка? не string.buf указатель на символ? в чем проблема с char *s="abcd"? Спасибо! :)

Answer 1

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

Что касается второго вопроса, то код в join() является фиктивным. Что вы делаете, это объявить локальную переменную, которая представляет собой массив из 256 символов. В качестве локальной переменной она будет «уничтожена», когда join() завершается.
Затем вы копируете s1 в этот буфер. Обратите внимание, что буфер может быть слишком мал, и вы получите переполнение буфера.

Затем вы вызываете strcat с локальным переменным буфером в качестве первого аргумента. Результатом является то, что s2 будет добавлен в s1 в локальном переменном буфере. Возврат значения strcat является указателем на буфер. Когда возвращается join(), буфер «уничтожается», и указатель становится недействительным.

Обратите внимание, что еще одна проблема в вашем коде - объявить функцию join() внутри основного. Вы должны перенести эту строку из основной функции.

То, как вы определили свою строковую и строковую переменную, верна. Это то, как вы выполняете join(), что является фиктивным.

Answer 2

Сообщение об ошибке полностью не требует пояснений. Непонятно, что вы не понимаете. Например, в вашей функции join произойдут ужасные вещи, если две строки будут длиннее 255 символов.

Также, ваша функция join полностью сломана. Он возвращает указатель на буфер, который больше не существует, как только он возвращается, поскольку вы выделили его в стек, из которого вы возвращаетесь.

Answer 3

Это your implementation trying to be helpful.

Вы можете остановить его от делать это, добавив следующий #define в верхней части кода

#define _CRT_SECURE_NO_WARNINGS 

или следовать предложению и использовать strcat_s().