Ошибка использования strcpy

//P6-05.CPP - макросредства для перменного списка параметров
 
#include<iostream> 
using namespace std;
#include<locale>
#include<sstream>  // Для работы со строками
#include<stdarg.h>  // Для макросредств
#include<cstdlib>  // Для функции malloc()
 
char *concat(char *s1, ...)
 
{
    va_list par;  // Указатель на параметры списка
    char *cp = s1;
    int len = strlen(s1); // Длина 1-го параметра
    va_start(par, s1); // Начало переменного списка
    // Цикл для определения общей длины параметров-строк:
    while (cp = va_arg(par, char *))
 
        len += strlen(cp); // Выделение памяти для результата:
    char *stroka = (char *)malloc(len + 1);
    strcpy(stroka, s1);
    va_start(par, s1);   // Начало переменного списка
    // Цикл конкатенации параметров строк:
    while (cp = va_arg(par, char *))
        strcat(stroka, cp); // Конкатенация двух строк
    va_end(par);
    return stroka;
    
}
 
int main()
{
    setlocale(LC_ALL, "Russian");
 
    char* concat(char* s1, ...); // Прототип функции
    char* s;                     // Указатель для результата
    s = concat("\nNulla ", "Dies ", "Sine ", "Linea!", NULL);
    s = concat(s, " - Ни одного дня без черточки!", "\n\t",
        "(Плиний Старший о художнике Апеллесе)", NULL);
    cout << s;
 
    cin.get();
    return 0;
}

strcpy_s(stroka, len, s1);

#define _CRT_SECURE_NO_WARNINGS