Подскажите, почему не работает strcpy_s()

@igorbelykh · Регистрация: 01.03.2013

Author24 — интернет-сервис помощи студентам

Вот код

C++

1	char *pcStr = "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.97 Safari/537.22"

C++

1 2	pUAgent = new char[strlen(pcStr)+1]; strcpy_s(pUAgent, strlen(pcStr), pcStr);

При компиляции ошибок не выдает. Но программы прекращает свое выполнение ошибкой именно на этой строке

C++

1	Программа "[4140] Test.exe" завершилась с кодом -1073740777 (0xc0000417).

alsav22 · 12.03.2013, 09:10

strcpy_s() - не стандарт. Используйте эту функцию:

C++

1	strncpy(pUAgent, pcStr, strlen(pcStr));

Tulosba · 12.03.2013, 11:43

Сообщение от alsav22

strcpy_s() - не стандарт.

То, что это поделка от MS не значит, что она не должна работать как надо.
Судя по описанию второй аргумент должен быть равен размеру приемного буфера, а не размеру копируемой строки (Size of the destination string buffer). Я бы вообще вынес strlen(pcStr)+1 в отдельную переменную и использовал ее и в new, и при копировании.
С strncpy() не стоит забывать, что может быть не добавлен \0 в конец строки, если размер исходной строки больше буфера. Что собственно и происходит, т.к. потеряна +1.

alsav22 · 12.03.2013, 12:09

Сообщение от Tulosba

С strncpy() не стоит забывать, что может быть не добавлен \0 в конец строки, если размер исходной строки больше буфера. Что собственно и происходит, т.к. потеряна +1.

Где тут что потеряно?

Tulosba · 12.03.2013, 12:19

Сообщение от alsav22

Где тут что потеряно?

Сравните варианты:

C++

#include <iostream>
#include <cstring>
 
using namespace std;
 
int main() {
   
   char dest[100];
   char* src = "0123456789";
   
   strncpy ( dest, src, strlen(src) ); // Вариант 1
   //strncpy ( dest, src, strlen(src)+1 ); // Вариант 2
   
   cout << dest << endl;
   
   return 0;
}

вывод.

Croessmah · 12.03.2013, 12:25

Сообщение от Tulosba

Судя по описанию второй аргумент должен быть равен размеру приемного буфера

Приемный буфер должен быть не меньше, чем исходная строка.

C++

#include <iostream>
 
void foo(const wchar_t * expression,const wchar_t * function,const wchar_t * file,unsigned int line,uintptr_t pReserved){
    std::cout<<"Invalid parameter"<<std::endl;
}
 
int main(){
    _set_invalid_parameter_handler(foo);
    _CrtSetReportMode(_CRT_ASSERT, 0);
    char * pcStr = "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.97 Safari/537.22";
   char * pUAgent = new char[strlen(pcStr)+1];
   strcpy_s(pUAgent, strlen(pcStr)+1, pcStr);//И без +1 попробуйте
}

Tulosba · 12.03.2013, 12:36

Сообщение от Croessmah

Приемный буфер должен быть не меньше, чем исходная строка.

Я о значении параметра, дословный перевод. Понятно, что если фактически буфер будет больше - ошибок это не увеличит. Кроме лишнего расхода памяти.

alsav22 · 12.03.2013, 12:40

Сообщение от Tulosba

Сравните варианты:

Я про код ТС с моим исправлением:

C++

1
2
3

char *pcStr = "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.97 Safari/537.22";
char *pUAgent = new char[strlen(pcStr)+1];
strncpy(pUAgent, pcStr, strlen(pcStr));

Где тут что потеряно?

Tulosba · 12.03.2013, 12:48

Сообщение от alsav22

Где тут что потеряно?

при вызове strncpy() не копируется последний \0-символ.
Чтобы это проверить добавьте м/у 2 и 3 строкой такую:

C++

1	memset( pUAgent, '?', strlen(pcStr)+1 );

Ну и посмотрите на вывод после strncpy(). Увидите лишний '?'.

alsav22 · 12.03.2013, 13:11

Тогда так:

C++

1
2
3

char *pcStr = "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.97 Safari/537.22";
char *pUAgent = new char[strlen(pcStr) + 1];
strncpy(pUAgent, pcStr, strlen(pcStr) + 1);

Или так:

C++

1
2
3

char *pcStr = "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.97 Safari/537.22";
char *pUAgent = new char[strlen(pcStr) + 1];
strcpy(pUAgent, pcStr);

Или так:

C++

char *pcStr = "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.22 (KHTML, like Gecko) Chrome/25.0.1364.97 Safari/537.22";
char *pUAgent = new char[strlen(pcStr) + 1];
strncpy(pUAgent, pcStr, strlen(pcStr));
pUAgent[strlen(pcStr)] = '\0';

@igorbelykh · 12.03.2013, 13:50 **[ТС]**

Всем спасибо. Исправил следующим образом и все заработало

C++

1 2	pUAgent = new char[strlen(pcStr)+1]; strcpy_s(pUAgent, strlen(pcStr)+1, pcStr);

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	12.03.2013, 11:43	3
	Сообщение от alsav22 strcpy_s() - не стандарт. То, что это поделка от MS не значит, что она не должна работать как надо. Судя по описанию второй аргумент должен быть равен размеру приемного буфера, а не размеру копируемой строки (*Size of the destination string buffer). Я бы вообще вынес strlen(pcStr)+1 в отдельную переменную и использовал ее и в new*, и при копировании. С strncpy() не стоит забывать, что может быть не добавлен \0 в конец строки, если размер исходной строки больше буфера. Что собственно и происходит, т.к. потеряна +1. 1

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	12.03.2013, 12:09	4
	Сообщение от Tulosba С strncpy() не стоит забывать, что может быть не добавлен \0 в конец строки, если размер исходной строки больше буфера. Что собственно и происходит, т.к. потеряна +1. Где тут что потеряно? 0

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	12.03.2013, 12:36	7
	Сообщение от Croessmah Приемный буфер должен быть не меньше, чем исходная строка. Я о значении параметра, дословный перевод. Понятно, что если фактически буфер будет больше - ошибок это не увеличит. Кроме лишнего расхода памяти. 0