problema funzione getcwd()

null · Messaggio da **null** » lun 18 ott 2004, 17:56

salve ragazzi, ho scritto il seguente codice:
 
 #include <stdio.h>
 
 int main(void){
 
 char *buff;
 size_t SIZE = 50;
 
 getcwd(buff,SIZE);
 
 printf("%s\n",buff);
 
 return 0;
 }
 e mi ha dato questo output in output prima la directory corrente (ed è quello che mi dovrebbe dare) e poi mi dice segmentation fault ( e non capisco il perchè). Qualcuno di voi saprebbe aiutarmi?
 Grazie

null · Messaggio da **null** » lun 18 ott 2004, 18:10

mi sono dimenticato una cosa:
 
 la funzione getwcd(char *BUFFER, size_t SIZE) memorizza in BUFFER la directory corrente, ma si dovrebbe anche specificare la grandezza SIZE del BUFFER.
 Ho letto che per evitare problemi di buffer overflow si potrebbe far puntare BUFFER a NULL e mettere SIZE a 0. Ma come si fa a far puntare BUFFER a NULL?

meritil · Messaggio da **meritil** » lun 18 ott 2004, 18:15

prova così:
 
 int main(void){
 
 char buff[50];
 size_t SIZE = 50;
 
 getcwd(buff,SIZE);
 
 printf("%s\n",buff);
 
 return 0;
 }
 
 oppure :
 
 int main(void){
 
 char * buff;
 size_t SIZE = 50;
 
 buff=(char*) malloc (sizeof(char) * 50);
 getcwd(buff,SIZE);
 
 printf("%s\n",buff);
 
 return 0;
 }
 
 Ciao

Messaggio da **useless** » lun 18 ott 2004, 18:27

soluzione migliore:
 
 #define SIZE 50
 
 int main(void){
 char buff[SIZE];
 
 getcwd(buff,SIZE);
 printf("%s\n",buff);
 
 return 0;
 }
 
 il tuo segfault è dovuto al fatto di non allocare una stringa (vettore di char), ma solo un puntatore, così quando la funzione getcwd prova a scrivere in memoria sovrascrive dati che non dovrebbe.
 
 <A HREF="http://www.danasoft.com" TARGET="_blank"><IMG SRC="http://www.danasoft.com/sig-ita.jpg"></A>
 Think free @ <A HREF="http://www.nyft.org" TARGET="_blank">http://www.nyft.org</A>
 [ Questo Messaggio è stato Modificato da: useless il 18-10-2004 18:27 ]

null · Messaggio da **null** » lun 18 ott 2004, 18:53

scusa useless potresti rispondere al mio secondo post

meritil · Messaggio da **meritil** » lun 18 ott 2004, 19:24

#include <stdio.h>
 #include <unistd.h>
 
 int main (int argc, char ** args){
 char * buff;
 
 if((buff=getcwd(NULL,0))==NULL) return EXIT_FAILURE;
 printf("%s\n",buff);
 free(buff);
 return EXIT_SUCCESS;
 
 }
 
 La versione posix di getcwd ti consente di non specificare la grandezza di buff e nemmeno di allocarlo fa tutto da solo attraverso una malloc wrappata dentro getcwd() chiaramente devi deallocare buff prima di uscire..
 In poche parole la stringa te la crea getcwd, a te serve solo un puntatore ad essa per utilizzarla... visto che te la crea getcwd con una malloc a te e demandato il compito di deallocarla. (con la free())
 
 Ciao Meritil
 
 ps:"Far puntare buff a NULL" significa passare NULL... [ Questo Messaggio è stato Modificato da: meritil il 18-10-2004 19:38 ]

null · Messaggio da **null** » lun 18 ott 2004, 19:31

ok meritil, ti ringrazio.
 Colgo anche l´occasione per farti vedere questo codice:
 
 #include <stdio.h>
 #include <stdlib.h>
 #include <dirent.h>
 
 int main(int argc, char *argv[]){
 
 struct dirent info,*d;
 DIR *dir;
 
 if(argc != 2){
 printf("ERRORE: uso %s dir-name\n",*argv);
 return 0;
 }
 
 if((dir = opendir(*++argv)) == NULL){
 printf("Non posso aprire la directory\n");
 return 0;
 }
 
 while(readdir_r(dir,&info,&d) == 0)
 printf("%s\n",d->d_name);
 
 closedir(dir);
 
 return 0;
 }
 
 Come output mi da la lista dei file che contiene la directory che gli passo come argomento, ma alla fine mi da sempre segmentation fault.
 Sapresti come aiutarmi e spiegrami il problema?

meritil · Messaggio da **meritil** » lun 18 ott 2004, 20:02

Allora io ti propongo come farei io...
 
 #include
 #include
 #include
 
 int main(int argc, char *argv[]){
 
 struct dirent *d;
 DIR *dir;
 
 if(argc != 2){
 printf("ERRORE: uso %s dir-name\n",*argv);
 return 0;
 }
 
 if((dir = opendir(args[1])) == NULL){
 printf("Non posso aprire la directory\n");
 return 0;
 }
 
 while((d=readdir(dir)) != NULL)
 printf("%s\n",d->d_name);
 
 closedir(dir);
 
 return 0;
 }
 
 
 Il tuo codice ora non ho modo di provarlo sono a casa di un amico con winz quindi niente compilatore

((
 Però a prima vista potrebbe essere l´ & prima di d essendo un puntatore puoi passarlo diretto, l´& posto prima passa il puntatore al puntatore a d... appena sono a casa provo e ti faccio sapere con certezza cmqe man readdir_r e controlla i tipi dei paramentri e te ne accorgi da solo dove sbagli...
 [ Questo Messaggio è stato Modificato da: meritil il 18-10-2004 20:04 ]

null · Messaggio da **null** » lun 18 ott 2004, 20:05

il problema è che con readdir(DIR *) funziona, ma conla funzione readdir_r(DIR *,struct dirent *,struct dirent **) non funziona.
 Ho messo &d proprio perchè vuole un puntatore a puntatore.

Messaggio da **useless** » mar 19 ott 2004, 0:33

il modo giusto x verificare se si è alla fine della dir o meno è:
 while(readdir_r(dir,&info,&d) == 0 && d)
 
 questo xké alla fine della directory d viene impostato a NULL, ma readdir_r ritorna ancora 0, e quindi come hai fatto tu la printf viene chiamata ancora una volta dopo l´ultimo elemento, passandole un puntatore a NULL, che genera il sigsegv.
 
 in realtà questo codice non tiene conto dei possibili errori che si possono verificare. se se ne verifica uno è come se la dir fosse finita.
 
 dove hai trovato sta funzione? nel mio man non c´è! c´è solo la readdir non thread-safe!
 
 <A HREF="http://www.danasoft.com" TARGET="_blank"><IMG SRC="http://www.danasoft.com/sig-ita.jpg"></A>
 Think free @ <A HREF="http://www.nyft.org" TARGET="_blank">http://www.nyft.org</A>
 [ Questo Messaggio è stato Modificato da: useless il 19-10-2004 00:34 ]

null · Messaggio da **null** » mar 19 ott 2004, 0:50

l´ho trovata in un manuale di C, si chiama "Imparare il C una guida per linux". E´ reperiribile in rete.
 comunque grazie ancora

null · Messaggio da **null** » mar 19 ott 2004, 0:58

scusa useless, mi sono dimenticato di chiederti come mai metti in && il puntatore d e non la stuttura info

Messaggio da **useless** » mar 19 ott 2004, 1:07

xké info ha un indirizzo ben preciso, locale a main, che viene stabilito quando lo dichiari:
 struct dirent info;
 
 successivamente readdir si limita a modificarne il contenuto, e ritorna il suo indirizzo in d, oppure ritorna NULL quando qualcosa è andato storto, e se non facesse questo non potresti accorgertene solo guardando il contenuto di info.
 
 se readdir dovesse modificare l´indirizzo di info avrebbe bisogno un ptr a un ptr.

meritil · Messaggio da **meritil** » mar 19 ott 2004, 3:46

Curioso che alla fine della directory imposti d a null ma restituisca cmqe 0 che a quanto leggo dal manuale significa "lettura eseguita con successo", cioè è come se dicesse "ho letto un cavolo con successo" eheh..
 Bah.. scriverlo in fondo al man risparmierebbe inutili giretti con gdb...
 
 Ciao.
 
 ps: Che ambiente usate per programmare in C ? Io sto cercando disperatamente CodeWarrior, su windows mi ci trovavo benissimo.. Sapete se esiste una cosa del genere per Linux?

Messaggio da **useless** » mar 19 ott 2004, 8:45

mah, in effetti hai ragione, xò mi sa che dobbiamo prendere 0 come "non c´è stato nessun errore". certo, è un po´ fuorviante, anche xké bisogna smenettare con errno x capire se è successo qualcosa o meno.
 poi comunque ribadisco che io nel man manco ce l´ho readdir_r

.
 
 quanto all´IDE: ho usato anjuta x un po´ (era l´unico programma x cui installavo le librerie di gnome!), poi sono passato a kdevelop, che un annetto fa mi aveva fatto letteralmente schifo, ma che da allora fortunatamente è molto migliorato.
 un´alternativa è eclipse, quel "coso" che si usa solitamente x programmare in java, x il quale xò esiste anche un plugin (CDT mi pare) x sviluppare in c/c++.
 altrimenti c´è sempre emacs

.