Dotaz: f-ce system v jazyce C

[Hanibal]

Zdravíčko, programuji jednoduchý časovač v jazyce C, přičemž používám f-ci system() a v ní příkaz shutdown. Mám ale problém vložit číslo z proměnné minuty do f-ce system, protože "%d" nebo %d není argumentem příkazu shutdown. Dá se s tím něco dělat? Děkuji

---

---

Odpovědi

[Ash]

char cmd[256];
sprintf(cmd, "shutdown %d", time);
result = system(cmd);

[jerry]

http://www.cplusplus.com/reference/clibrary/cstdio/sprintf

Použij sprintf na vytvoření řetězce a ten dej jako parametr do system()

[Hanibal]

Děkuju

[Michal Kubeček]

Funkce system() je poměrně nebezpečná hračka a její neuvážené použití snadno vytváří bezpečnostní chyby. Nemám moc dobrý pocit, když ji vidím používat někoho, kdo se potřebuje ptát na takhle elementární obraty.

[Václav Vanc]

Můžeš uvést nějaké odstrašující příklady, abych věděl, na co si mám dávat pozor?

[David Watzke]

No snadno tak někomu můžeš předat shell aniž bys chtěl. Někde ho necháš zadat řetězcovej argument pro nějakej příkaz, třeba "cmd %s" a on tam prdne něco jako "; /bin/bash", atd. Tohle je například problém u programů, které se spouštějí se sudo.

[Filip Jirsák]

A od bezmyšlenkovitého použití %d je k %s sakra blízko. Stačí, že si autor vzpomene, že chce použít plné možnosti zadání času pro shutdown.

[martin-ux]

I ked nie je presne zname co autor riesi, pripada mi vhodnejsie pouzitie skriptu na dany problem. Ked uz C, tak zavolat reboot nativne.

Ale ked uz zacala diskusia o security, tak by bolo vhodne spomenut aj fakt, ze miesto sprintf by mal autor pouzit snprintf.

[martin-ux]

xjo .. to mala byt reakcia na autora threadu, nie na teba michal.

[Ash]

Ale ked uz zacala diskusia o security, tak by bolo vhodne spomenut aj fakt, ze miesto sprintf by mal autor pouzit snprintf.

Jenže to by se pak muselo také říct proč, a to by bylo na dlouho, jinak by to při jeho úrovni znalostí vedlo k tomu, že by použil

char cmd[256];
snprintf(cmd, 256, "shutdown %d", time);
result = system(cmd);

což je nesmysl a spíš bych čekal že si pak kus toho čísla ustřihne a shutdown se provede v jiný čas, což by mohlo být větší bezpečnostní riziko :) strncpy není něco jako "bezpečná verze strcpy" v oblasti security, spíš brání nechtěným chybám v oblasti dočasné programátorovy stupidity která občas potká každého z nás. I když je nepochybně mnoho případů, kdy je vhodné ji použít, hlavně proto, aby se člověk musel zamyslet, jak dlouhý ten vstup vlastně může být. Pokud se ovšem zamyslí a řekne si, že nejlepší bude tam dát strlen(vstup) a má to teda bezpečné, protože to psali na abclinuxu, tak jsme tam, jde jsme byli :D

[martin-ux]

ani nie ze brani stupidite, skor ju limituje .. ale ma zmysel, kedze nuti sa zamysliet co vlastne kopirujem a hlavne kolko ..

neber to ale ako kritiku tvojho prispevku .. sam som hore napisal, ze nevieme co autor chce .. ale ked uz bol naznak o hovoreni bezpecnostnych chyb, tak stoji za zmienku aj to, co som napisal (aspon tak si myslim ja ).

[Ash]

Jasně, já to neberu jako kritiku, popravdě já spíš kritizuji :D Když jsem psal tu odpověď, tak jsem nejprve napsal ze zvyku strncpy, pak si říkám, tady jsme asi na jiném levlu, a napsal jsem to jednoduššeji. Protože kdybych řekl A (že strncpy), musel bych říct i B a C (proč, jak). Podle mne to také určitě stojí za zmínku, nejlépe takovou, která by rozebrala problém víc do hloubky, nebo alespoň dobrý odkaz... Bez toho mi to připadá že to nebude mít očekávaný efekt, přestože to je připomínka jistě fakticky správná a k věci.

[Jardík]

Všechno s fixně velikým bufferem je kravina použít tam, kde se fixně veliký buffer nehodí, např. tady. Jednoho dne si to někdo bude chtít zkompilovat na plafromě, kde maximální hodnota intu jako řetězec bude mít 250 znaků a bum. Fce snprintf() nám přece vrátí potřebný počet znaků a pokud je jaksi >= než co jsme měli alokováno, můžeme přealokovat. Bohužel nějakek kokot si řekl, že bude vracet int, ale malloc() chce size_t (a funkce snprintf taky), takže bude následovat kýbl maker či podmínek, kde se porovnává jejich velikost k určení, zda bude typecast bezpečný. Taky vás pak omezují na řetězec s maximální délkou MIN(SIZE_MAX-1, INT_MAX-1), což je stupidní.

Závěr? snprintf nepoužívat, tato funcce je k ho*nu a podobně i další z její rodiny.

[Ash]

Tak pokud se pohybujete v takové teoretické rovině kde int má víc než 250 znaků, tak můžete rovnou použít nějakou GNU funkci printf která vám ten string naalokuje a nepotřbujete si předalokovávat nic, žejo.

[Jardík]

GNU funkci

A v tom je háček, zrušíte tak přenositelnost programu a pak nemá teda ani cenu zabývat se velikým intem, budete na tom stejně.

[AraxoN]

Ehm... int, ktorý môže mať v desiatkovom vyjadrení 250 a viac číslic, musí mať najmenej 831 bitov. Povedal by som, že takýto dátový typ nám v kategórii "primitívne dátové typy" tak skoro nehrozí. :)

[Jardík]

Jenže pak předpokládáte něco, co standard nevylučuje a jsme tam, kde jsme byli na začátku. Pak si ještě řekneme, že pravděpodobně na nic jiného, než 32bit int nenarazíme a že nám bude stačit 10 znaků pro <-2^31;2^31-1>, použijeme %d s 11 znakovým bufferem (pro ukončovací nulu) a pak zjistíme, že nám to chtělo dát "-2,125,256,111" protože nějaký locale.

[Ash]

No uvažovat 64bit nebo 830bit je trochu rozdíl :D Ale základní myšlenku chápu.

[Pindal]

K prdu je dnes myslet na 250 znaků int.

[Nikola Pajkovský]

char cmd[sizeof("shutdown %d") + sizeof(unsinged) * 3];
sprintf(cmd, "shutdown %d", time);
...
...

[Michal Kubeček]

:-)

[Ash]

překladač snese všechno

[Michal Kubeček]

Ten překlep asi ne. Ale jinak je to zajímavá myšlenka, jen pro velké hodnoty sizeof(unsigned) to bude zbytečné plýtvání. :-)

[Ash]

Zvlášť pro ty 830bitové :)