видя сбой приложения при попытке использовать strcpy

Question

видя сбой приложения при попытке использовать strcpy

0

У нас есть cpp dll, где я мог сужаться до линии, откуда возникала авария. Строка связана с копией, которая:

void GetCellText(HWND ssHwnd,char *& output){
  CString sData;
  ....
  ....
  strcpy (output, sData) //app crashes here
}

поэтому я заменил strcpy следующим образом:

void copyToOutput(char *& output, CString sData) {
  int strLen = sData.GetLength();
  output = (char *) malloc(sizeof(char) * strLen + 1); // Allocate memory
  LPTSTR p = sData.GetBuffer(strLen);
  strcpy(output, p);
  output[strLen] = '\0'; // Null terminate
  sData.ReleaseBuffer();
}

однако я все еще вижу отказ от сбоев. Не могли бы вы кого-нибудь предложить, что неправильно с моим исправлением.

user1328572 27 авг. 2014, в 08:01

Источник

0

Является ли output указывающий на выделенную память, достаточно большим, чтобы вместить все в sData ? И sData не строка широких символов?
Some programmer dude 27 авг. 2014, в 05:35
1

О, и есть ли причина, по которой вы используете указатель в стиле C на строку вместо, например, CString (или std::string ) для аргумента? Так как функция в любом случае вызывается из кода C ++, вы также можете использовать правильные строковые классы.
Some programmer dude 27 авг. 2014, в 05:40
0

Отслеживание стека вашего сбоя будет более информативным, чем все, что мы можем догадаться о вашем коде здесь. Несвязанный: передать sData по ссылке на вашу функцию; нет смысла делать больше копий этой вещи, чем нужно. Мое предположение : вы развращаете свою кучу где-то еще.
WhozCraig 27 авг. 2014, в 07:53
0

правильно, из вызывающей функции есть строка free (output), которая вызывает сбой. Но моя новая проблема заключается в том, что вызываемая функция имеет определенные потоки, где мой метод вообще не вызывается, то есть нет malloc, я думаю, что это вызывает свободный сбой. Но как проверить, была ли выделена память или память или нет ??
user1328572 27 авг. 2014, в 10:16

Показать ещё 2 комментария

Теги:

c++

dll

c

2 ответа

Ещё вопросы

Является ли output указывающий на выделенную память, достаточно большим, чтобы вместить все в sData ? И sData не строка широких символов?
О, и есть ли причина, по которой вы используете указатель в стиле C на строку вместо, например, CString (или std::string ) для аргумента? Так как функция в любом случае вызывается из кода C ++, вы также можете использовать правильные строковые классы.
Отслеживание стека вашего сбоя будет более информативным, чем все, что мы можем догадаться о вашем коде здесь. Несвязанный: передать sData по ссылке на вашу функцию; нет смысла делать больше копий этой вещи, чем нужно. Мое предположение : вы развращаете свою кучу где-то еще.
правильно, из вызывающей функции есть строка free (output), которая вызывает сбой. Но моя новая проблема заключается в том, что вызываемая функция имеет определенные потоки, где мой метод вообще не вызывается, то есть нет malloc, я думаю, что это вызывает свободный сбой. Но как проверить, была ли выделена память или память или нет ??

Maxim Egorushkin · Answer 1 · 2014-08-27T07-20-00.000Z

В этом коде может быть несколько проблем:

Никогда не используйте strcpy поскольку он подвержен переполнению буфера назначения. В вашем случае вы знаете длину скопированной строки, нет причин не использовать memcpy, например memcpy(output, p, strLen).
Это C++ код, но буфер выделяется с помощью malloc. Вызывающий этот код может случайно использовать free[] чтобы освободить его. Вы можете использовать std::vector<char> вместо необработанного указателя.
Вы можете напрямую обращаться к базовой C-строке без необходимости ее копирования с помощью CString::PCXSTR.

user2522767 · Answer 2 · 2014-08-27T05-51-00.000Z

Используйте это и решите сбой.

output=(char *) malloc(sData.GetLength());
strcpy(output,sData.GetString() );

из вызывающей функции есть строка free (output), которая вызывает сбой. Но моя новая проблема заключается в том, что вызываемая функция имеет определенные потоки, где мой метод вообще не вызывается, то есть нет malloc, я думаю, что это вызывает свободный сбой. Но как проверить, была ли выделена память или память?
Это приведет к сбою, потому что он не выделяет достаточно памяти для вывода.