Dotaz: Segmentation fault při kopírování argv[]

[Snur]

Zdravím. Vytvářím v céčku aplikaci která bude číst vstup přes argumenty zadané v příkazové řádce. Mám však problém s kopírováním parametrů z argv[] do připraveného pole. Ovšem tištění platných parametrů problém nedělá. Zde je část zdrojového kódu:

#include < stdio.h >
#include < stdlib.h >
#include < string.h >

int main(int argc, char * argv[])
{
	char argvtemp[100];
	strcpy(argvtemp,argv[1]);
        .
        .
        .

Při spuštění dělá toto:

~$ ./program --parametr="Super věc"
Neoprávněný přístup do paměti (SIGSEGV)

Když se na to podívám přes gdb tak uvidím toto:

Breakpoint 1, main (argc=1, argv=0xbf864434) at desccreator.c:8
8               strcpy(argvtemp,argv[1]);
(gdb) step
Program received signal SIGSEGV, Segmentation fault.
0xb7e22cc0 in strcpy () from /lib/libc.so.6

A když napíši tohle namísto strcpy():

printf("%s\n",argv[1]);

přeložím a spustím, dostanu normálně:

~$ ./program --parametr="Super věc"
--parametr="Super věc"

Nevíte někdo co s tím? Už si opravdu nevím rady. Připadá mi to na hlavu postavené, strcpy() přece z argv[] jen čte stejně jako printf(), tak proč u kopírování dochází k chybě a u tištění ne? Do programu takhle načítám data poprvé takže možná že dělám někde chybu. Budu vděčný za každou radu.

Odpovědi

[#Tom]

Nevidím důvod, proč by měl program padat, když je mu zadán parametr kratší než 99 znaků, což se v tomto případě děje, i tak je v něm ale chyba. Nelze dopředu zaručit, že bude nastaven jiný než první prvek pole, počet parametrů programu udává parametr argc funkce main. Dále je třeba dát pozor na délku řetězce.

char *p;
…
if (argc >= 1) {
  p = malloc(strlen(argv[1])+1);
  if (p != NULL) {
    strcpy(p, argv[1]);
  }
}
…

[#Tom]

Měl bych si dávat větší pozor, v podmínce má být samozřejmě argc >= 2.

[Semo]

Netusim, cim je to sposobene, ale vsimni si vypis breakpointu:

Breakpoint 1, main (argc=1, argv=0xbf864434) at desccreator.c:8

argc=1 a ty sa snazis citat z argv[1].

[Messa]

Ten kód je funkční (ale v pořádku není, viz předchozí příspěvky). Spíš si myslím, že v tom programu máš ještě jednu chybu, která vyvolá SIGSEGV, i když program spustíš s parametrem (nebo leda že by sis ten výpis po spuštění vymyslel), ale když program debuguješ, zapomeneš tam ten parametr dát a program slítne hned na strcpy

[pakanek]

Me ten tvuj kod hazi SIGSEGV jen pokud jej spustim bez parametru. Pokud jej spustim s parametrem, tak je program v poradku.

[Snur]

Máte všichni pravdu.U spouštění programu jsem na argumenty nezapomínal, ovšem během ladění v gdb už ano. Proto ten argc=1, což jsem si měl taky všimnout. Takže během všelijakých úprav se zdrojovým kódem jsem dostávál zkreslené informace o pravém původci chyby což v mém případě byla malá chyba v jedné funkci kterou jsem si napsal a chtěl rychle otestovat, proto ten nedotažený kód. Ale jak vidno měl jsem ho přece v tomto případě dotáhnout, dříve bych přišel kde dělám chybu. Taková stupidní chyba - připadám si jak blbec. Velice díky za rady.