Hahaha
Comunque ti rinnovo l’augurio di buon lavoro, se diverrà un bel progetto potrei essere uno dei futuri utenti

Ero già a conoscenza di tutti i problemi che mi hai segnalato. Come ho già detto, non avevo nemmeno l’intenzione di pubblicare qualcosa fino a quando il progetto non fosse stato più maturo.

Il trimmer attuale è solo il risultato di una urgenza personale avuta una notte di due mesi fa. Quando l’ho scritto non mi sono preoccupato troppo dei controlli necessari nel codice né di eventuali altri problemi secondari. Ho seguito la semplice filosofia “It just works for me”

Già il fatto di controllare se il file è una rom valida solo guardando alla sua estensione la dice lunga

A dirla tutta, la funzione di trimming è solo la punta dell’iceberg. Ho intenzione di implementare in questa utility la gestione completa dell’header dei file NDS, magari aggiungendo tra le altre cose anche la possibità di estrarre i file in essi contenuti. Ma questo si vedrà col tempo, e se ce ne sarà reale utilità.

L’unica cosa certa è che questa utility resterà indipendente dal gestore “a la amarok” che ho in mente: da buon gentooista quale sei, sarai certamente daccordo con me se dico che non c’è nulla di meglio di un buon tool CLI per essere sicuri che un lavoro sia svolto bene

Per il resto, la penso allo stesso modo: la scimmietta è mitica

Come seconda cosa, ho dato un’occhiata veloce al codice, ben scritto e ben commentato, ma c’è un bel problemino di buffer overflow, alle righe 216 e 217. Il problema è che se il nome del file supera i 511 caratteri il buffer viene semplicemente ecceduto. Converrebbe modificare il codice, utilizzando strncpy al posto di strcpy, in modo da troncare la stringa, qual’ora troppo lunga, e poi aggiungere il carattere nullo, se dovesse servire. La linea 216 diverrebbe 2 linee, e verrebbe così:

strncpy(backupname, argv[i], 511 - strlen(”.bak”));
backupname[511] = 0;

Poi, se non vuoi troncare, potresti usare la heap, ma già diventa un po’ più complicato. Oppure, se non t’interessa la massima compatibilità col POSIX, potresti usare asprintf:

asprintf(backupname, “%s.bak”, argv[i]);

…

free(backupname);

In questo modo allocheresti backupname della dimensione necessaria, automaticamente. Bisogna solo ricordarsi di deallocarla alla fine, quando non serve più. asprintf è disponibile con le glibc, e sotto *BSD, perciò abbastanza diffusa, ma non la trovi proprio ovunque.

Poi, per essere pignoli, dovresti controllare che almeno le funzioni più importanti vadano a buon fine, e dovresti anche mettere un link ad un file di testo che contiene l’intera GPL, ma insomma, quelli son dettagli.

In ogni caso buon lavoro e grazie

Ah, e
Mi piaceva la faccina