Что делать со вторым параметром GetCurrentDirectory(int, LPTSZTR ZdesKosak)

[Mike_I]

Привет народ.
Исользую функцию GetCurrentDirectory() в VC.net
Первый ее параметр размер буфера, второй сам указатель на буфер, в буфере
после выполнения находится путь к рабочей директории приложения.
Вроде все работает путь возвращается правильно, но после его получения
с текстовым буфером ничего нельзя сделать, ни записать в него ни даже удалить!!!

Код:
try
{
  CString strPath;

  strPath = "";// Здесь все инициализируется нормально
  dwExecutionResult = GetCurrentDirectory( 55, strPath.GetBuffer() );
 
  strPath = "123";// Здесь выскакивает ошибка
}// Если закоментить предыдущую строку то ошибка выскочит когда придем сюда
catch(......

[Джон]

Mike_I,  лучше конечно понимать, что делаешь. Ты передаёшь в ф-ю пустой буфер, а говоришь, что он у тебя размером 55. Ф-ции по барабану - ты отвечаешь за буфер. Практически она пишет по указателю, который ты ей передал данные. Ессно с переполнением буфера. Вот пошёл ты скажем за пивом, даёшь продавцу рюмку и говоришь: "Налейте мне десять литров." Что получится?

Ты должен буфер сначала подготовить, а потом передать указатель на него в ф-ю GetCurrentDirectory. Откуда у тебя это магическое число 55? А если длинна пути будет 56, или 80, или 100? Поэтому надо передавать буфер с запасом. В таких случаях лучше  пользоваться константами, в данном случае
MAX_PATH, например так:

Код:

	char chCurDir[MAX_PATH];
	memset(chCurDir,0,MAX_PATH);
	::GetCurrentDirectory(MAX_PATH,chCurDir);

[Anchorite]

Должно быть так:
   

Код:

try {
    DWORD dwBufferLength = ::GetCurrentDirectory(0, NULL);
    CString strPath;

    dwExecutionResult = GetCurrentDirectory(dwBufferLength, strPath.GetBuffer(dwBufferLength));
    strPath.ReleaseBuffer();
}
catch(......

[Mike_I]

Джон:
Но за буфер, я думаю, случае должны отвечать функции класса CString, на то он и есть. К тому же даже если буфер неверен к тому что переменная закрывается для использования, это отношения не имеет. По идее должна просто стерется часть нужной инфы. Это все-равно что продавец тебе налил 3 литра пива в литровую банку, 2 соответственно вылилось, но фишка в том что банка при этом прилипла к прилавку и ее не унести, с недостаточным ее объемом это не связано.

[Mike_I]

Anchorite:
 Если сделать как ты говоришь то ошибка выскакивает на строке
  strPath.ReleaseBuffer();

[Finch]

Mike_I, По умолчанию в объекте выделена память под буфер 16 байт. Тем более ты иницилизируеш так

Код:

strPath = "";

В функцию ты отдаеш указатель на данный буфер, который находится в объекте. WinAPI функция не знает, и знать не хочет, что ты ей подсовываеш. Она думает что ты даеш указатель на  массив символов размером 55 байт. Естественно честно пишет в эти 55 байт. И естествено полностью уничтожает все данные самого экземпляра объекта строки и прилежаших с ней переменных. Классический пример ошибки переполнения буффера.

[Джон]

Mike_I, ты думаешь не правильно (в данном случае), поэтому ещё раз призову тебя сначала разобраться с материей, прежде чем делать выводы (ошибочные).

Дело в том, что в ф-ю GetCurrentDirectory НЕ передаётся объект CString. Если бы это было так, тогда конечно не надо было бы передавать размер - объект сам позаботился бы о размере своего буфера. А так передаётся указатель типа char, и как же, скажи пожалуйста, через него ф-я GetCurrentDirectory должна управлять объектом CString? Может придумаешь?

А посему повторю ещё раз (в последний - если не поймёшь, я уж и не знаю) - это не проблема объекта CString, а ф-ции GetCurrentDirectory. Ты сам должен позаботиться о размере буфера, как впрочем при работе практически с любой ф-ей API, запрашивающей буфер для ввода данных.
В примере Anchorite указание объекту создать сначала буфер необходимого размера даётся с помощью параметра ф-ции GetBuffer, который он запросил у GetCurrentDirectory, вызвав её с параметром NULL.

А с буфером всё происходит так как я объяснил, а не так как тебе хочется. Ты создаёшь объект strPath и обнуляешь его strPath = ""; Вызови GetLength и убедись, что длинна равна нулю.
Потом ты передаёшь указатель на кусок памяти размером 0 (заключающий нуль я не считаю) в ф-ю и говоришь напиши мне туда 55 байт. Даже если запрашиваемый путь всего 3 символа "c:\" - уже происходит ошибка всех новичков и дилетантов (залог успешной работы всех хакеров) - переполнение буфера. В дебаге ты можешь это и не заметить. Всё! 55 литров пива вылиты в банку с нулевым объёмом. Твоя память в сортире, какие могут быть последствия не скажет тебе никто. Даже ГБ и БГ.
Тебе повезло, что вылетает в след строчке - ошибку сразу видно. У меня подобная ошибка приводила к тому, что сбой происходил в совершенно другом месте, при совершенно другой операции. И найти подобные ошибки в большом проекте ой как тяжело. Так что учи мат. часть.

[Джон]

зы По причине жуткой нехватки времени - отвечал почти целый день. За это время пришли ещё сообщения.

Anchorite показал направление в котором надо работать. Ведь весь твой код мы не видим. А так ещё надо учитывать размерность символа - например в UNICODE он равен не одному, а двум байтам. CString свободно обходтся с подобными вещами и возвращает число символов, а не байт. Поэтому правильно использовать так - dwBufferLength*sizof(TCHAR)
Это раз. Два - размер буфера в GetBuffer(dwBufferLength) не учитывает заключительный ноль. И если GetCurrentDirectory прописывает его сама - получаем опять переполнение буфера.
И наконец третье - перед тем как вызывать Release ты должен убедится, что никто не работает с буфером.

[Mike_I]

Джон:
 А как мне тогда инициализировать strPath если буфер должен быть допустим 100 символов писать что-то типа:
 strPath = "123456789012345678901234567890...и так еще 70 раз";

[Джон]

Если так хочется работать с API и MFC:

Код:

CString GetCurDir()
{
	CString stOut=_T("");
	DWORD dwLen = ::GetCurrentDirectory(0,NULL);
	TCHAR *pszCurDir = new TCHAR[dwLen+1];
	memset(pszCurDir,0,dwLen*sizeof(TCHAR));
	::GetCurrentDirectory(dwLen,pszCurDir);
	stOut = CString(pszCurDir);
	delete [] pszCurDir;
	return stOut;
}

зы ф-я работает с UNICODE и с ANSI (второе не проверял - у меня ща UNICODE проект)

[Джон]

Джон:
 А как мне тогда инициализировать strPath если буфер должен быть допустим 100 символов писать что-то типа:
 strPath = "123456789012345678901234567890...и так еще 70 раз";

Mike_I, ну ну это уже национаглость - ты мои посты вобще не читаешь?
Для подобных случаев есть константы - длинна пути ограничена в винде - я думал, что это общеизвестный факт.

В таких случаях лучше  пользоваться константами, в данном случае
MAX_PATH, например так:

Код:

	char chCurDir[MAX_PATH];
	memset(chCurDir,0,MAX_PATH);
	::GetCurrentDirectory(MAX_PATH,chCurDir);

Использование константы особенно рекомендовано при компиляции на разных платформах кажется под 9х длинна равна 256, а под виндой 2 КБ (ща точно не помню)