Dotaz: popen, pclose a C

[kafi]

Preji pekny vecer naprogramoval jsem softik v cecku, na mem ubuntu jede bez problemu. Kdyz jsem ho nahral na server (slackware 9.1) kam ho potrebuju tak se vyskytla hlaska pri spousteni externich prikazu Broken pipe.A v tu chvili cely soft spadne a je konec. Uz jsem googlil co se dalo ale nikde jsem nic nenasel nevite nekdo kde by mohla byt chyba? prikladam dve fce ve ktereych se to deje. teda spis si myslim ze to je problem toho Slackwaru nez meho programovani. preklad provadim pomoci g++ "zdroj.cpp" -o "vystup". Hlaska se objevi po returnu z fce.

unsigned int exec_command (const char *p_command)
{
	FILE *pipe_cmd=NULL;
	
	if ((pipe_cmd=popen(p_command,"w"))==NULL){
		fprintf(stderr,"Don't open pipe!\n");
		exit(1);
	}
	fputs(p_command, pipe_cmd);
	pclose(pipe_cmd);
	return 0;
}

nebo pro cteni

unsigned int read_command (const char *p_command) 
{
	FILE *pipe_cmd=NULL;
	char p_cmd[50]=("cat /var/log/syslog | grep "), vystup[2];

	strcat (p_cmd, p_command);

	if ((pipe=popen(p_cmd,"r"))==NULL) {
		fprintf(stderr,"Nelze spustit externi prikaz!\n");
		exit (1);
	}
	
	if ((fgets(vystup, 2, pipe))!=NULL)  {
		pclose (pipe);
		return 1;
	}
	else {
		pclose (pipe);
		return 0;
	}
}

Odpovědi

[Luboš Doležel (Doli)]

Ručně jde ten příkaz spustit bez problémů? Pokud jde o můj systém, tak tu žádný /var/log/syslog není.

Nebylo by lepší nahradit veškeré spouštění podprocesů vlastním kódem?

[kafi]

Hm to v mem pripade pujde tezko spoustim i iptables a to prez cecko asi neporesim. a ja proste potrebuju cist vystup toho prikazu. a prikaz system vraci jen jestli se provedlo ci nikoliv.

[Luboš Doležel (Doli)]

A nebude k takovým věcem lepší Bash nebo Perl?

[kafi]

Prave premyslim o nejhorsim, ze bych 600 radku poceni prepsal do perlu, jak i vi pisete, ale kur.. jak se mi nechce. takove sklamani jsem z cecka dlouho nezazil.

[Michal Kubeček]

Nemluvte, prosím, o zklamání z céčka. Céčko nemůže za to, že jste si nepřečetl dokumentaci.

[Michal Kubeček]

Chcete-li číst výstup toho příkazu, pak druhý parametr popen musí být "r", ne "w". Hláška "broken pipe" znamená, že se pokoušíte zapisovat do roury, jejíž druhý konec mezitím druhá strana zavřela. V tomto případě příkaz iptables skončí, aniž by četl to, co mu posíláte na stadardní vstup.

[petris]

Existuje a ma spravna prava soubor /var/log/syslog? Jsou vsechny spoustene programy v $PATH?

[qk]

No zaprvy bych rekl, ze neni tohle zrovna idealni zpusob progamovani..navic mi to prijde jako dost nesikovny.
Chyby co vidim -
nekontrolujes jestli se povedlo fputs.
osobne bych taky dal aspon aserty na vstupni parametry.

Jinak ten error ti rika, ze se pokousis cist nebo zapisovat do pipe, kterou uz nekdo na druhe strane zavrel (at uz zapisovatel nebo ctenar).

[kafi]

puts nekontroluju protoze jsem to testoval. jestli jde aspon spoustet kdyz ne cist. vstupni parametry tech funkci jsou osetreny pri volani tech f-ci. takze v tele samotnych f-ci mi to prijde vcelku zbytecne. jinak dekuji za reakce. stale nehazim flintu do zita a experimentuju. prepisovat do perlu se mi to fakt nechce.

[kafi]

Jeste jsem zapomnel. jak uz jsem psal chybove hlaseni se objevi po returnu z fce. takze je zadouci aby jiz bykla roura zavrena. takze jsem z toho lehce zmateny. proc neni zavrena?

[qk]

Ale to broken pipe dle mne vyhodi to fputs nebo fgets...ne az to fclose....schvalne doporucuju pro ladeni tam dat nejake vypisy.

[petris]

Co ta práva? Zkousel jste koukat do strace?

[lkfjsu984]

Jak uz tady qkecko rikalo, Broken pipe znamena, ze zapisujes do pipy, ktera uz je zavrena. Jde totiz o to, ze kdyz pustis ten program tak je to vicemene samostatna vec, ktera nejak komunikuje s tim tvym programem. A jde o to, jak ma ten program poznat jestli jeste ten druhy neskocil, je pripraveny cist a tak. Komunikace se dela prostrednictvim signalu a konkretne v tomhle pripade asi SIGPIPE. A ty signaly muzes bud ignorovat (jak to delas) nebo je odchytavat a pak treba kdyz ho dostanes tak proste skoncit program misto dalsiho psani do neexistujici pipy. (man 7 signal, ...). Ja to obvykle resim:

void pipebreak(int sig)
{
}
...
signal(SIGPIPE, pipebreak);

(tohle ovsem neni priklad hodny nasledovani).

Nicmene podle me tam je nejaka nekonzistence v tom co ty spoustene prikazy vypisuji (asi jina verze na obou pocitaich) a tak to bude asi potreba nejak osetrit.

[Jirka P]

(tohle ovsem neni priklad hodny nasledovani).

Podle mně je to naprosto v pohodě (i když mnohem lepší ignorovat je pomocí signal(SIGPIPE, SIG_IGN) (pozn. default je terminate). Potom se ta chyba projeví tak že fputs selže a v errno je EPIPE (tuším).

Každopádně na ladění takových chyb je nejlepší strace -f

[kafi]

ak panove driv jsem se k tomu nedostal az ted a koul jsem se na to co stim a zjistil jsem:

1/ spousteny command jsem skladal do pole deklarovaneho jako cmd[300]
2/ udelal jsem z nej dynamicky aloccovane a najednou zacal fungovat i sprintf
3/ ve vstupech do funkci jsem udelal z pointeru na command staicky command[300]

a funguje.