C++: Как избежать ввода пользователем для безопасных системных вызовов?

Question

На платформе Linux, у меня есть C++ код, который идет как это:

// ... 
std::string myDir; 
myDir = argv[1]; // myDir is initialized using user input from the command line. 
std::string command; 
command = "mkdir " + myDir; 
if (system(command.c_str()) != 0) { 
    return 1; 
} 
// continue.... 

• Проходит вход пользователя в систему() называют безопасным на всех?
• Следует ли избегать/дезинформировать пользователя?
• Как?
• Как этот код может быть использован для вредоносных целей?

Спасибо.

Answer 1

Просто не используйте system. Предпочитают execl.

execl ("/bin/mkdir", "mkdir", myDir, (char *)0); 

Таким образом, myDir всегда передается как один аргумент в mkdir, а оболочка не участвует. Обратите внимание, что если вы используете этот метод, вам понадобится fork.

Но если это не просто пример, вы должны использовать функцию mkdir C:

mkdir(myDir, someMode); 

Answer 2

Использование системы() вызов с параметрами командной строки без дезинфицирующего ввода может быть весьма небезопасно.

Потенциальная угроза безопасности может быть пользователь проходит следующее в качестве имени каталога

somedir ; rm -rf/

Чтобы избежать этого, используйте смесь следующего

• использования Getopt, чтобы обеспечить ввод в санированном
• дезинфицировать ввод
• использовать execl вместо системы для выполнения запятая й

Наилучшим вариантом было бы использовать все три

Answer 3

В дополнение к ответу от Матфея, не породит процесс оболочки, если вы абсолютно не нужно. Если вы используете комбинацию fork/execl, отдельные параметры никогда не будут проанализированы, поэтому их не нужно избегать. Остерегайтесь нулевых символов, однако это еще преждевременно прекратит параметр (в некоторых случаях это не проблема безопасности).

Я предполагаю, что mkdir является просто примером, поскольку mkdir можно тривиально вызывать из C++ гораздо легче, чем эти предложения подпроцесса.