Как правильно закрывать popen?

внутри. если вдруг ты попытаешься закрыть свой попен через указатель, принимающий значение NULL, то твоему попену может стать весьма неуютно.

внутри. если вдруг ты попытаешься закрыть свой попен через указатель, принимающий значение NULL

Спасибо.

Я вот почему спрашиваю: если в строке - FILE *curl_prog = popen(«curl http://192.168.5.189 2> /dev/null», «r»);, указать неправильную команду, например так - FILE *curl_prog = popen(«murl http://192.168.5.189 2> /dev/null», «r»);, то всё равно возвращается != NULL, то есть получается, что открытие произошло?

Может всё-таки использовать api curl, а не попе-н?

Может всё-таки использовать api curl, а не попе-н?

Допустим нужно запустить другую команду. Речь не про curl, а про popen.

man 3 popen

NULL вернется, если функция не может выделить память. При ошибках нужно смотреть errno.

NULL вернется, если функция не может выделить память.

Я поторюсь.

Если в строке:

FILE *curl_prog = popen(«curl http://192.168.5.189 2> /dev/null», «r»);

Допустить ошибку:

FILE *curl_prog = popen(«Murl http://192.168.5.189 2> /dev/null», «r»);

То возвращается НЕ NULL - это что означает, поток открылся или не открылся?

я повторюсь. это ничего не означает, кроме того, что память не выделилась, надо смотреть errno. (и читать мануалы)

Оно у тебя в любом случае откроется, тк popen() вызывает (обычно) /bin/sh, а ему передаёт уже строку, которую ты написал. Если shell не сможет выполнить команду, это не значит, что popen() зафейлился.

надо смотреть errno.

perror(«Error»); говорит - Error: Success

Если shell не сможет выполнить команду, это не значит, что popen() зафейлился.

Дык и я про то же, но как это обрабатывать?

Как правильно закрывать popen?

Как pclose(). Обрабатывать самое простое - перенаправлять в самой команде stderr в stdout и вычитывать ошибку из канала. Ну и ЕМНИП pclose вернёт тебе код возврата последней команды, посмотри man для верности.

Как pclose().

Я имел в виду это:

char curl_buff[64] = {0,};

FILE *curl_prog = popen("curl http://192.168.5.189 2> /dev/null", "r");

if(curl_prog != NULL)
 {
   fgets(curl_buff, 63, curl_prog);
 }

if(pclose(curl_prog) == -1) error_log("close curl_prog.");

pclose делать после фигурных скобок или внутри?

внутри

ОК

1. Если у тебя ресурсов достаточно, и всё в порядке с шеллом пользователя popen всегда вернёт годный указатель.
2. После безошибочного pclose он вернёт статус завершения команды.

Если команда левая, можно после первого чтения из файла проверить что EOF, закрыть если да, и проверить возврат pclose.
Ещё раз, в нормальных условиях popen вернёт валидный указатель.

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

Ну вот зачем ты спойлеришь? Так весело было читать..

Popen() возвращает NULL только одном случае, если не смог сделать fork(). А fork() на линуксе редко возвращает ошибку. Очень редко.

В строчке могут быть хоть стихи написаны, если fork() удался, popen() вернёт не NULL.

Чтобы получить реальную ошибку запуска, надо сначала прочитать весь поток до EOF-а, и только после этого pclose() вернет ошибку запуска.

Popen() возвращает NULL только одном случае, если не смог сделать fork()

Спасибо.

Закрытие должно быть после фигурных скобок или внутри...?

http://pubs.opengroup.org/onlinepubs/007904875/functions/pclose.html

If the argument stream to pclose() is not a pointer to a stream created by popen(), the result of pclose() is undefined.

Так что внутри. Ну или не внутри, главное — спроверкой на NULL.

Popen() возвращает NULL только одном случае, если не смог сделать fork().

http://pubs.opengroup.org/onlinepubs/007904875/functions/popen.html

The popen() function may fail if:

[EMFILE]
{FOPEN_MAX} or {STREAM_MAX} streams are currently open in the calling process.
[EINVAL]
The mode argument is invalid.