15.10.2006 17:17
[#1]
[C, VBCC] suspicious format string
Ciągle otrzymuję z kompilatora taki warning "suspicious format string" przy takich linijkach kodu:
i podobnych.
Wcześniej do
Dlaczego dla kompilatora VBCC ten string jest "suspicious"?
W sumie jest to tylko warning. Jak dotąd wszystko działa tak jak chcę i nic mi się nie zwiesiło, ale to jest denerwujące. Poza tym może mój program działa tylko przypadkiem :D i ten warning jest rzeczywiście uzasadniony?
sprintf(buffer, "%.*s", lsize, buffer);
i podobnych.
bufferz tego przykładu mam zdefiniowany jako
char *, a
lsizejako
int.
Wcześniej do
bufferwczytuję treść pliku tekstowego otwartego w trybie
"r".
Dlaczego dla kompilatora VBCC ten string jest "suspicious"?
W sumie jest to tylko warning. Jak dotąd wszystko działa tak jak chcę i nic mi się nie zwiesiło, ale to jest denerwujące. Poza tym może mój program działa tylko przypadkiem :D i ten warning jest rzeczywiście uzasadniony?
16.10.2006 08:41
[#6]
Re: [C, VBCC] suspicious format string
@R-Tea, post #1
hmm, wydaj mi się że jest błąd w twoim kodzie,
Przecież oczekujesz stringa( char *) a masz wpisane lsize które jest intem jak powiedziałeś a dopiero później masz buffer i dlatego kopilator cosik tam podejrzewa że go chcesz w bambuko zrobić. W zasadzie to nie wiem po co jest ten lsize . Chyba powinno być tak.
Pozdrawiam
sprintf(buffer, [b]"%.*s"[/b], lsize, buffer);
Przecież oczekujesz stringa( char *) a masz wpisane lsize które jest intem jak powiedziałeś a dopiero później masz buffer i dlatego kopilator cosik tam podejrzewa że go chcesz w bambuko zrobić. W zasadzie to nie wiem po co jest ten lsize . Chyba powinno być tak.
sprintf( buffer, "%.*s", buffer );
Pozdrawiam
16.10.2006 20:51
[#7]
Re: [C, VBCC] suspicious format string
@Kaczus, post #5
@ Kaczuś
No napisalem bez nawiasow kwadratowych, zeby bylo widac.
hej, ale mnie chodzi o mój kod w C, a uwagi dotyczące taga "code" to proszę do grx'a zgłaszać :)
mam takie coś:
sprintf(buffer, "%.*s", lsize, buffer);
lsize to int
buffer to char*
VBCC ciągle mnie ostrzega, że to jest suspicious format string. Co jest źle?
No napisalem bez nawiasow kwadratowych, zeby bylo widac.
hej, ale mnie chodzi o mój kod w C, a uwagi dotyczące taga "code" to proszę do grx'a zgłaszać :)
mam takie coś:
sprintf(buffer, "%.*s", lsize, buffer);
lsize to int
buffer to char*
VBCC ciągle mnie ostrzega, że to jest suspicious format string. Co jest źle?
16.10.2006 21:13
[#8]
Re: [C, VBCC] suspicious format string
@asman, post #6
@ asman
sprintf(buffer, "%.*s", lsize, buffer);
ta gwiazdka oznacza, że w tym miejscu będzie podany maksymalny rozmiar pola, w tym przypadku lsize.
Faktycznie, bez gwiazdki i lsize się nie czepia, ale to mnie nie ratuje, bo gdy nie określę maksymalnego rozmiaru pola to funkcja wypisująca teść na ekran czyta poza buforem i mam czasem syf na ekranie, a czasem zwieszki.
I jeszcze jedna dziwna rzecz, a mianowicie ta kropka przed gwiazdką. Nie wiem co ona robie (znalazłem taki przykład w internecie), ale bez tej kropki mam taki sam efekt jakbym w ogóle nie określił rozmiaru pola, czyli też przypadkowy syf i zwieszki.
sprintf(buffer, "%.*s", lsize, buffer);
ta gwiazdka oznacza, że w tym miejscu będzie podany maksymalny rozmiar pola, w tym przypadku lsize.
Faktycznie, bez gwiazdki i lsize się nie czepia, ale to mnie nie ratuje, bo gdy nie określę maksymalnego rozmiaru pola to funkcja wypisująca teść na ekran czyta poza buforem i mam czasem syf na ekranie, a czasem zwieszki.
I jeszcze jedna dziwna rzecz, a mianowicie ta kropka przed gwiazdką. Nie wiem co ona robie (znalazłem taki przykład w internecie), ale bez tej kropki mam taki sam efekt jakbym w ogóle nie określił rozmiaru pola, czyli też przypadkowy syf i zwieszki.
16.10.2006 22:00
[#12]
Re: [C, VBCC] suspicious format string
@Kaczus, post #9
@ Kaczuś
Pewny jestes, że jest to w standardzie?
jest o tym w "Język ANSI C" Kerninghama, choć cokolwiek inaczej objaśnione:
"Specyfikacja przekształcenia steruje przekształceniem kolejnego pola wejściowego. Zwykle wynik jest przypisywany zmiennej wskazanej odpowiednim argumentem. Jeśli jednak przypisanie ma być wstrzymane, jak np. w %*s, do dane pole wejściowe będzie po prostu pominięte - nie wykona się żadne przypisanie"
niż to znalazłem w necie:
http://cplusplus.com/ref/cstdio/sprintf.html
tabelka width
Pewny jestes, że jest to w standardzie?
jest o tym w "Język ANSI C" Kerninghama, choć cokolwiek inaczej objaśnione:
"Specyfikacja przekształcenia steruje przekształceniem kolejnego pola wejściowego. Zwykle wynik jest przypisywany zmiennej wskazanej odpowiednim argumentem. Jeśli jednak przypisanie ma być wstrzymane, jak np. w %*s, do dane pole wejściowe będzie po prostu pominięte - nie wykona się żadne przypisanie"
niż to znalazłem w necie:
http://cplusplus.com/ref/cstdio/sprintf.html
tabelka width
17.10.2006 08:00
[#13]
Re: [C, VBCC] suspicious format string
@R-Tea, post #8
ta gwiazdka oznacza, że w tym miejscu będzie podany maksymalny rozmiar pola, w tym przypadku lsize.
Faktycznie, bez gwiazdki i lsize się nie czepia, ale to mnie nie ratuje, bo gdy nie określę maksymalnego rozmiaru pola to funkcja wypisująca teść na ekran czyta poza buforem i mam czasem syf na ekranie, a czasem zwieszki.
Faktycznie jest z tą gwiazdką tak jak mówisz określa rozmiar pola, ale bez tego czyta poza buforem, może potrzeba jakiegoś konkretnego końca , tylko nie wiem jakiego ( przeważnie w C jest nim zero, ale może w tym przypadku potrzeba EOF ). Moja propozycja jest taka, wywalić gwiazdkę i lsize a zaalokować troszkę więcej pamięci na buffer i dodać zero na końcu i sprawdzić albo dodać EOF na końcu i też sprawdzić . Powinno być dobrze.
A byłbym zapomniał dwa linki może będą pomocne
http://www.thinkage.ca/english/gcos/expl/c/lib/sprint.html
http://www.thinkage.ca/english/gcos/expl/c/lib/printf.html
Pozdrawiam
Ostatnia modyfikacja: 17.10.06 08:05
Faktycznie, bez gwiazdki i lsize się nie czepia, ale to mnie nie ratuje, bo gdy nie określę maksymalnego rozmiaru pola to funkcja wypisująca teść na ekran czyta poza buforem i mam czasem syf na ekranie, a czasem zwieszki.
Faktycznie jest z tą gwiazdką tak jak mówisz określa rozmiar pola, ale bez tego czyta poza buforem, może potrzeba jakiegoś konkretnego końca , tylko nie wiem jakiego ( przeważnie w C jest nim zero, ale może w tym przypadku potrzeba EOF ). Moja propozycja jest taka, wywalić gwiazdkę i lsize a zaalokować troszkę więcej pamięci na buffer i dodać zero na końcu i sprawdzić albo dodać EOF na końcu i też sprawdzić . Powinno być dobrze.
A byłbym zapomniał dwa linki może będą pomocne
http://www.thinkage.ca/english/gcos/expl/c/lib/sprint.html
http://www.thinkage.ca/english/gcos/expl/c/lib/printf.html
Pozdrawiam
Ostatnia modyfikacja: 17.10.06 08:05
17.10.2006 22:34
[#14]
Re: [C, VBCC] suspicious format string
@asman, post #13
@ asman
Moja propozycja jest taka, wywalić gwiazdkę i lsize a zaalokować troszkę więcej pamięci na buffer i dodać zero na końcu i sprawdzić albo dodać EOF na końcu i też sprawdzić.
hmmm.. wg Kernighana i Ritchiego EOF jest typu int. Jak mam dokleić inta na koniec bufora tyou char* ?
Próbowałem strcat() ale wychodzą hocki klocki. Zresztą strcat() oczekuje const char* jako drugiego argumentu, a nie int.
Zajrzałem jeszcze do stdio.h i tam EOF jest zdefiniowany w ten sposób:
#define EOF (-1)
Moja propozycja jest taka, wywalić gwiazdkę i lsize a zaalokować troszkę więcej pamięci na buffer i dodać zero na końcu i sprawdzić albo dodać EOF na końcu i też sprawdzić.
hmmm.. wg Kernighana i Ritchiego EOF jest typu int. Jak mam dokleić inta na koniec bufora tyou char* ?
Próbowałem strcat() ale wychodzą hocki klocki. Zresztą strcat() oczekuje const char* jako drugiego argumentu, a nie int.
Zajrzałem jeszcze do stdio.h i tam EOF jest zdefiniowany w ten sposób:
#define EOF (-1)
18.10.2006 00:32
[#16]
Re: [C, VBCC] suspicious format string
@R-Tea, post #1
A jaki właściwie efekt chcesz uzyskać?
Z tego co widzę, to z tego samego do tego samego stringa chcesz skopiować nie więcej niż lsize znaków.
Ale przecież można prościej (i szybciej):
Pamiętam, że kiedyś też mi się vbcc czepiał czegoś (ten sam komunikat), ale działało. No ale sam komunikat mówi, że to jest tylko 'podejrzane', może dlatego, że taka konstrukcja może być wynikiem pomyłki i kompilator przed nią ostrzega, tak samo jak przed:
Z tego co widzę, to z tego samego do tego samego stringa chcesz skopiować nie więcej niż lsize znaków.
Ale przecież można prościej (i szybciej):
if(strlen(buffer)>lsize) buffer[lsize] = 0;
Pamiętam, że kiedyś też mi się vbcc czepiał czegoś (ten sam komunikat), ale działało. No ale sam komunikat mówi, że to jest tylko 'podejrzane', może dlatego, że taka konstrukcja może być wynikiem pomyłki i kompilator przed nią ostrzega, tak samo jak przed:
if(a=b)