преобразование bash (например, ssh) в c ++ с использованием std :: system

Question

преобразование bash (например, ssh) в c ++ с использованием std :: system

0

Я пытаюсь преобразовать свой код bash в код c++. У меня есть что-то вроде этого:

ssh ${USER}@${HOSTS[0]} "bash -c \"cd ${PROGRAM_PATH}; svn update; rm -rf build; make\""

Как я могу преобразовать это в c++ с помощью std :: system? Я попробовал это:

const std::string bashFunc("bash -c \"cd " + k_programPath + "; svn update; rm -rf build; make\"");
const std::string func(getSSH(k_user, server) + " \"" + bashFunc + "\"");
std::system(func.c_str());

Когда я печатаю func, все выглядит хорошо. Но когда я запускаю его, он не соединяется с ssh и только пытается выполнить команды bash в моей локальной папке.

EDIT: Это выглядит так:

ssh username @server "bash -c" cd./path/to/folder; обновление svn; rm -rf build; делать""

gartenriese 25 нояб. 2014, в 00:03

Источник

0

Наверное, плохая идея. svn есть библиотека, и есть OpenSSL и т.д ..
Basile Starynkevitch 24 нояб. 2014, в 23:10
0

Что плохого в использовании bash? И вы могли бы напечатать фактическую строку, которую вы сформулировали?
Mats Petersson 24 нояб. 2014, в 23:16
0

@Basile Starynkevitch: Для меня это небольшой побочный проект, ничего сложного.
gartenriese 24 нояб. 2014, в 23:17
0

@ Матс Петерссон: Я хочу добавить многопоточность, потому что одна команда ssh за другой просто занимает слишком много времени. Я отредактировал свой вопрос.
gartenriese 24 нояб. 2014, в 23:20
1

И замена оригинальной строки на make -j N где N - это подходящее число, не работает?
Mats Petersson 24 нояб. 2014, в 23:28
0

Для этого вам не нужно использовать многопоточность - вы можете использовать фоновые процессы в native bash и получать параллелизм с гораздо меньшей сложностью.
Charles Duffy 24 нояб. 2014, в 23:33
0

Кроме того, передача строки в вызов system() чрезвычайно подвержена ошибкам по сравнению с выполнением вызова execve() с явным массивом аргументов.
Charles Duffy 24 нояб. 2014, в 23:34
0

@ Матс Петерссон: Это не то, что я хочу сделать. Я вызываю ssh на многих серверах, чтобы проверить, запущена ли конкретная программа, но проверка> 100 серверов на ssh по очереди занимает слишком много времени.
gartenriese 25 нояб. 2014, в 00:52
0

@Charles Duffy: Сначала я хотел попробовать параллелизм с bash, но это выглядело довольно сложно, поэтому я решил использовать нити C ++, потому что, по крайней мере, я их знаю :)
gartenriese 25 нояб. 2014, в 00:53
0

Если вы задали вопрос о специфическом аспекте параллелизма в bash, который доставлял вам неприятности, возможно, мы могли бы помочь с этим. Серьезно - я обещаю, что это будет проще и менее подвержено ошибкам, чем вызов связки оболочек из C ++.
Charles Duffy 25 нояб. 2014, в 01:04
0

(Кроме того, использование конкатенации строк для замены переменных в ваших оболочках приводит к тому, что вам нужно беспокоиться об уязвимостях внедрения оболочки; ваш код будет гораздо более безопасным, если вы передадите строковые литералы через среду, а не подставите их в код).
Charles Duffy 25 нояб. 2014, в 01:06
0

@ Чарльз Даффи: Хорошо, я мог бы просто спросить. Не больно знать оба пути. Не беспокойтесь о безопасности, это просто быстрый и грязный частный проект, ничего для официального релиза;)
gartenriese 25 нояб. 2014, в 01:11
0

@Charles Duffy: Вот мой вопрос по Bash.
gartenriese 25 нояб. 2014, в 01:19

Показать ещё 11 комментариев

Теги:

c++

bash

ssh

2 ответа

1

Не используйте system() в любых, но наиболее простых случаях - он добавляет дополнительный слой анализа синтаксиса, каждый из которых увеличивает количество цитирования и экранирования, участвующих в вашем коде. Вместо этого используйте члена семейства вызовов execv():

char * const command[] = {"bash", "-c", "yourcommand", NULL};
execv("/usr/bin/bash", command);

Перед достижением этого кода убедитесь, что fork() вы не скрываете выход из сценария и wait() в родительском процессе, если хотите, чтобы он прочитал статус выхода своего дочернего элемента.

Charles Duffy 24 нояб. 2014, в 22:34

0

Спасибо за предложение, я попробую это. Вместо fork () я буду использовать std :: async.
gartenriese 25 нояб. 2014, в 01:10
1

На самом деле это обязательно, что вы используете fork() . Попытка сделать его встроенным с помощью std::async() приведет к завершению вашего процесса при первом вызове exec * -series.
Charles Duffy 25 нояб. 2014, в 01:13
0

Это неудачно!
gartenriese 25 нояб. 2014, в 01:14

Показать ещё 1 комментарий

Ещё вопросы

Наверное, плохая идея. svn есть библиотека, и есть OpenSSL и т.д ..
Что плохого в использовании bash? И вы могли бы напечатать фактическую строку, которую вы сформулировали?
@Basile Starynkevitch: Для меня это небольшой побочный проект, ничего сложного.
@ Матс Петерссон: Я хочу добавить многопоточность, потому что одна команда ssh за другой просто занимает слишком много времени. Я отредактировал свой вопрос.
И замена оригинальной строки на make -j N где N - это подходящее число, не работает?
Для этого вам не нужно использовать многопоточность - вы можете использовать фоновые процессы в native bash и получать параллелизм с гораздо меньшей сложностью.
Кроме того, передача строки в вызов system() чрезвычайно подвержена ошибкам по сравнению с выполнением вызова execve() с явным массивом аргументов.
@ Матс Петерссон: Это не то, что я хочу сделать. Я вызываю ssh на многих серверах, чтобы проверить, запущена ли конкретная программа, но проверка> 100 серверов на ssh по очереди занимает слишком много времени.
@Charles Duffy: Сначала я хотел попробовать параллелизм с bash, но это выглядело довольно сложно, поэтому я решил использовать нити C ++, потому что, по крайней мере, я их знаю :)
Если вы задали вопрос о специфическом аспекте параллелизма в bash, который доставлял вам неприятности, возможно, мы могли бы помочь с этим. Серьезно - я обещаю, что это будет проще и менее подвержено ошибкам, чем вызов связки оболочек из C ++.
(Кроме того, использование конкатенации строк для замены переменных в ваших оболочках приводит к тому, что вам нужно беспокоиться об уязвимостях внедрения оболочки; ваш код будет гораздо более безопасным, если вы передадите строковые литералы через среду, а не подставите их в код).
@ Чарльз Даффи: Хорошо, я мог бы просто спросить. Не больно знать оба пути. Не беспокойтесь о безопасности, это просто быстрый и грязный частный проект, ничего для официального релиза;)
Спасибо за предложение, я попробую это. Вместо fork () я буду использовать std :: async.
На самом деле это обязательно, что вы используете fork() . Попытка сделать его встроенным с помощью std::async() приведет к завершению вашего процесса при первом вызове exec * -series.

gartenriese · Accepted Answer · 2014-12-22T23-06-00.000Z

В дополнение к экранированию кавычек мне также пришлось скрывать косые черты, как это

const std::string bashFunc("bash -c \\\"cd " + k_programPath + "; svn update; rm -rf build; make\\\"");

Это работает сейчас.