CreateProcess() - некорректно передаются параметры командной строки

@Sasha · Регистрация: 05.06.2008

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

Пытаюсь открыть файл через CreateProcess(), в параметрах прописал путь к программе которая будет открывать файл и путь к самому файлу. Но почему-то Блокнот открывает не тот файл, который я прописал в пути, а создаёт новый.

C++

STARTUPINFO cif;
ZeroMemory(&cif,sizeof(STARTUPINFO));
PROCESS_INFORMATION pi;
if(CreateProcess(L"C:\\windows\\notepad.exe",L"E:\\Sniffer\\Win32\\Release\\Log.txt", NULL, NULL, false, NULL, NULL, NULL, &cif,&pi))
{
    ShowMessage("Норм");       
 }      
else         
 {      
        ShowMessage(SysErrorMessage(GetLastError()));                 
}

Как делать так чтобы блокнот открывал именно тот файл, который я ему прописал в параметре функции?

@D1973 · 09.02.2015, 17:16

Ну, вот так как-то...

C++

void __fastcall TForm1::Button1Click(TObject *Sender)
{
  STARTUPINFO si;
  PROCESS_INFORMATION pi;
  ZeroMemory( &si, sizeof(si) );
  si.cb = sizeof(si);
  ZeroMemory( &pi, sizeof(pi) );
  if(!CreateProcess(NULL, L"notepad.exe C:\\TEMP\\Unit1.txt",NULL,NULL,FALSE,0,NULL,NULL,&si,&pi))
    {
        ShowMessage(String().sprintf(L"CreateProcess failed (%d).", GetLastError()));
        return;
    }
    WaitForSingleObject( pi.hProcess, INFINITE );
    CloseHandle( pi.hProcess );
    CloseHandle( pi.hThread );
}

Убежденный · 09.02.2015, 19:17

Добавлю, что Unicode-версии функций CreateProcess могут модифицировать
аргумент, через который передается командная строка. Поэтому вместо

C

//
// Здесь L"MyCommandLine" - неизменяемая (read-only) строка.
//
CreateProcess(NULL, L"MyCommandLine", ...);

следует писать как-то так:

C

//
// А здесь, напротив, буфер CmdLine можно перезаписывать,
// поэтому нижеследующий фрагмент кода безопасен.
//
wchar_t CmdLine[] = {L"MyCommandLine"};
CreateProcess(NULL, CmdLine, ...);

@Avazart · 09.02.2015, 23:31

Интересно только в чем эта модификация может заключаться, и чем может вылиться подстановка строковых типов с const_cast<>

C++

        
std::wstring  cmdLine_;
CreateProcess( appName_.empty()?   0: appName_.c_str(),
                                     cmdLine_.empty()? 0: const_cast<wchar_t*>(cmdLine_.c_str()), // ???
                                     &security_,
                                     &security_,
                                     true,
                                     priority_,
                                     0,
                                     currentDir_.empty()? 0: currentDir_.c_str(),
                                     &startupInfo_,
                                     &processInfo_);

Убежденный · 10.02.2015, 00:02

Видел (в исходниках Win2K), что CreateProcessW на этапе разбора аргументов
временно ставит ноль между именем exe и остальной частью командной строки.
Судя по коду, чтобы "не заморачиваться" с выделением дополнительного буфера и
иметь возможность вызывать SearchPath, CreateFileW и другие функции для
различных проверок. Короче, передавать строковой литерал, как и любую другую
константную строку, в lpCommandLine чревато, и это легко проверяется и
явно обозначено в MSDN, кстати. Подчеркну: здесь речь идет именно о реальной
константности, т.е. когда передается read-only буфер. У std::string.c_str()
на известных мне реализациях C++ константность чисто формальная.

Ну а почему с CreateProcessA таких проблем нет ? Да потому, что она всего лишь
обертка над CreateProcessW (как и подавляющее большинство A-функций), которая
выделяет временные буферы на время вызова CreateProcessW и поэтому там
access violation не происходит.

@Avazart · 10.02.2015, 00:06

Не по теме:

Я просто видел в исходниках QProcess (Qt) там вообще кастуют в си стиле.

Убежденный · 10.02.2015, 00:27

Не по теме:

Ну вряд ли разработчики Windows-бэкенда Qt настолько глупцы, чтобы не читать MSDN :)
Скорее всего, там ведь чисто "синтаксическая" константа. Я тоже видел, как string.c_str()
передается как read-write аргумент: это, хоть и вызывает легкий озноб, до AV не доводит.

@Sasha · 06.10.2015, 10:02 **[ТС]**

По ходу сделал вот так вот

C++

           STARTUPINFO si;
    PROCESS_INFORMATION pi;
    ZeroMemory(&si, sizeof(si));
    si.cb = sizeof(si);
    ZeroMemory(&pi, sizeof(pi));
    wchar_t CmdLine[] = {L"/source=folder /pri=2 /path=C:\\DATA\"};
    if (!CreateProcess(NULL, CmdLine, NULL, NULL, FALSE, 0, NULL, NULL, &si, &pi))
    {
        ShowMessage(SysErrorMessage(GetLastError()));
        return;
    }
    WaitForSingleObject( pi.hProcess, INFINITE );
    CloseHandle( pi.hProcess );
    CloseHandle( pi.hThread );

Но выдаёт ошибку

[bcc32 Error] Unit1.cpp(292): E2034 Cannot convert 'wchar_t[37]' to 'char *'
[bcc32 Error] Unit1.cpp(292): E2342 Type mismatch in parameter 'lpCommandLine' (wanted 'char *', got 'wchar_t *')

Использую XE4. Не могу понять, здесь, что конкретно надо char? Если да то почему wchar_t нельзя здесь использовать?

@volvo · 06.10.2015, 10:54

В XE2 этот код прекрасно компилируется (если убрать лишний слеш из строки, разумеется)

@Sasha · 06.10.2015, 11:10 **[ТС]**

volvo, где именно ты убрал лишний слешь?

@volvo · 06.10.2015, 11:11

В 6-ой строке, последний слеш. Что, по подсветке не видно, что он просто экранирует кавычку? Ты либо его убери, либо еще один добавь, смотря что нужно.

@Sasha · 06.10.2015, 11:21 **[ТС]**

Странно, убирал и добавлял эффект тот же

@volvo · 06.10.2015, 11:29

Значит, проверяй, что в настройке "_TCHAR maps to" в Project->Options->Directories and Conditionals. Если там выставлен char - то у меня выдает точно ту же ошибку, о которой ты говоришь. Поменяй на wchar_t и пересобери проект.

@Avazart · 06.10.2015, 11:30

Не по теме:

Sasha,
А если пальцы в розетку в суть? Не пробовал? Может мозг включится? Любители потыкать наугад, блин ...

Добавлено через 1 минуту

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

C++

1	wchar_t CmdLine[] = {L"/source=folder /pri=2 /path=C:\\DATA"};

Кстати что это?

@Sasha · 06.10.2015, 18:51 **[ТС]**

volvo, спасибо, помогло

Не по теме:

Avazart, Хамить не обязательно!!!!!!!!!!!!!!!!!!!!

@Sasha 4956 / 2420 / 531 Регистрация: 05.06.2008 Сообщений: 7,518 Записей в блоге: 3
	06.10.2015, 11:21 [ТС]	12
	Странно, убирал и добавлял эффект тот же 0

@volvo Супер-модератор 32835 / 21172 / 8148 Регистрация: 22.10.2011 Сообщений: 36,431 Записей в блоге: 8
	06.10.2015, 11:29	13
	Значит, проверяй, что в настройке "_TCHAR maps to" в Project->Options->Directories and Conditionals. Если там выставлен char - то у меня выдает точно ту же ошибку, о которой ты говоришь. Поменяй на wchar_t и пересобери проект. 0

Убежденный Ушел с форума 16473 / 7436 / 1187 Регистрация: 02.05.2013 Сообщений: 11,617 Записей в блоге: 1
	10.02.2015, 00:02	5
	Видел (в исходниках Win2K), что CreateProcessW на этапе разбора аргументов временно ставит ноль между именем exe и остальной частью командной строки. Судя по коду, чтобы "не заморачиваться" с выделением дополнительного буфера и иметь возможность вызывать SearchPath, CreateFileW и другие функции для различных проверок. Короче, передавать строковой литерал, как и любую другую константную строку, в lpCommandLine чревато, и это легко проверяется и явно обозначено в MSDN, кстати. Подчеркну: здесь речь идет именно о реальной константности, т.е. когда передается read-only буфер. У std::string.c_str() на известных мне реализациях C++ константность чисто формальная. Ну а почему с CreateProcessA таких проблем нет ? Да потому, что она всего лишь обертка над CreateProcessW (как и подавляющее большинство A-функций), которая выделяет временные буферы на время вызова CreateProcessW и поэтому там access violation не происходит. 3

@Avazart
	10.02.2015, 00:06 #6
	Не по теме: Я просто видел в исходниках QProcess (Qt) там вообще кастуют в си стиле. 0

Убежденный
	10.02.2015, 00:27 #7
	Не по теме: Ну вряд ли разработчики Windows-бэкенда Qt настолько глупцы, чтобы не читать MSDN :) Скорее всего, там ведь чисто "синтаксическая" константа. Я тоже видел, как string.c_str() передается как read-write аргумент: это, хоть и вызывает легкий озноб, до AV не доводит. 0

@volvo Супер-модератор 32835 / 21172 / 8148 Регистрация: 22.10.2011 Сообщений: 36,431 Записей в блоге: 8
	06.10.2015, 10:54	9
	В XE2 этот код прекрасно компилируется (если убрать лишний слеш из строки, разумеется) 0

@Sasha 4956 / 2420 / 531 Регистрация: 05.06.2008 Сообщений: 7,518 Записей в блоге: 3
	06.10.2015, 11:10 [ТС]	10
	volvo, где именно ты убрал лишний слешь? 0

@volvo Супер-модератор 32835 / 21172 / 8148 Регистрация: 22.10.2011 Сообщений: 36,431 Записей в блоге: 8
	06.10.2015, 11:11	11
	В 6-ой строке, последний слеш. Что, по подсветке не видно, что он просто экранирует кавычку? Ты либо его убери, либо еще один добавь, смотря что нужно. 0

@Sasha 4956 / 2420 / 531 Регистрация: 05.06.2008 Сообщений: 7,518 Записей в блоге: 3
	06.10.2015, 18:51 [ТС]	15
	volvo, спасибо, помогло Не по теме: Avazart, Хамить не обязательно!!!!!!!!!!!!!!!!!!!! 0

CreateProcess() - некорректно передаются параметры командной строки

Решение