[C] Creare una shell

[-hide-]

Ciao ragazzi.
Ho iniziato da poco a seguire il corso di Sistemi Operativi pensando a come studiare al meglio questa materia divertendomi un pò.
L'idea è quella di creare una shell ma non trovo abbastanza materiale googlando. Sapete darmi qualche informazione?!

[fero86]

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

int main() {
	char *psz = NULL;
	size_t cb;
	do {
		free(psz);
		getline(&psz, &cb, stdin);
		system(psz);
	} while (strcmp(psz, "exit"));
	return 0;
}

[-hide-]

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

int main() {
	char *psz = NULL;
	size_t cb;
	do {
		free(psz);
		getline(&psz, &cb, stdin);
		system(psz);
	} while (strcmp(psz, "exit"));
	return 0;
}

Questa dichiarazione, ed il suo successivo utilizzo, a che serve?!

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

int main() {
	char *psz = NULL;
	size_t cb;
	do {
		free(psz);
		getline(&psz, &cb, stdin);
		system(psz);
	} while (strcmp(psz, "exit"));
	return 0;
}

Quest'altra istruzione invece?

Inoltre mi da questo errore, che roba è?

Undefined symbols:
  "_getline", referenced from:
      _main in ccQsPdgz.o
ld: symbol(s) not found
collect2: ld returned 1 exit status

[bobbytre]

Originariamente inviato da -hide-

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

int main() {
	char *psz = NULL;
	size_t cb;
	do {
		free(psz);
		getline(&psz, &cb, stdin);
		system(psz);
	} while (strcmp(psz, "exit"));
	return 0;
}

Questa dichiarazione, ed il suo successivo utilizzo, a che serve?!

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

int main() {
	char *psz = NULL;
	size_t cb;
	do {
		free(psz);
		getline(&psz, &cb, stdin);
		system(psz);
	} while (strcmp(psz, "exit"));
	return 0;
}

Quest'altra istruzione invece?

Inoltre mi da questo errore, che roba è?

Undefined symbols:
  "_getline", referenced from:
      _main in ccQsPdgz.o
ld: symbol(s) not found
collect2: ld returned 1 exit status

size_t è un tipo , più precisamente un intero senza segno , e viene usato per le dimensioni es. di un file , o di un buffer.

system è una funzione che fa eseguire al sistema operativo il comando passato.

per risolvere il problema che ti da getline , prova a sostituire la funzione , usa gets

[DanieleC88]

Originariamente inviato da fero86

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

int main() {
	char *psz = NULL;
	size_t cb;
	do {
		free(psz);
		getline(&psz, &cb, stdin);
		system(psz);
	} while (strcmp(psz, "exit"));
	return 0;
}

Azz, che fai, una free() su un puntatore a NULL? KA-BOOM!

[fero86]

Originariamente inviato da DanieleC88

Azz, che fai, una free() su un puntatore a NULL? KA-BOOM!

tratto da http://linuxmanpages.com/man3/free.3.php :

free() frees the memory space pointed to by ptr, which must have been returned by a previous call to malloc(), calloc() or realloc(). Otherwise, or if free(ptr) has already been called before, undefined behaviour occurs. If ptr is NULL, no operation is performed.

[fero86]

Originariamente inviato da -hide-

Questa dichiarazione, ed il suo successivo utilizzo, a che serve?!

é solo per dare qualcosa alla getline. la documentazione non dice che li posso mettere NULL (almeno non mi pare).

Quest'altra istruzione invece?

come sarebbe a dire, non mi dire che non hai mai fatto system("PAUSE") per evitare che la console del programma si chiuda subito

Inoltre mi da questo errore, che roba è?

chiedo scusa, mi accorgo solo ora che la getline é un'estensione GNU.
scrivere una shell in C é molto difficile a causa del solo idiotissimo compito di leggere un'intera linea dallo standard input: il 90% del codice é quello
perció te la riscrivo in C++ perfettamente standard:

#include <iostream>
#include <cstdlib>

using namespace std;

int main() {
	while (true) {
		string strLine;
		getline(cin, strLine);
		if ((strLine == "exit") || cin.eof()) {
			return 0;
		}
		system(strLine.c_str());
	}
}

[fero86]

piccolo perfezionamento:

#include <iostream>
#include <cstdlib>

using namespace std;

int main() {
	while (true) {
		cout << "> ";
		string strLine;
		getline(cin, strLine);
		if ((strLine == "exit") || cin.eof()) {
			return 0;
		}
		system(strLine.c_str());
	}
}

cosi stampa anche un prompt

[Teo@Unix]

una shell in C (ottimo esercizio secondo me).

Si può fare in vari modi, potresti anche pensare ad un modello client-server... una shell remota insomma forse è ancora meglio come esercizio.

Dovresti darti un occhio al multiplexing in C. Nonchè alla programmazione di rete.

[maulattu]

Originariamente inviato da bobbytre

size_t è un tipo , più precisamente un intero senza segno , e viene usato per le dimensioni es. di un file , o di un buffer.

system è una funzione che fa eseguire al sistema operativo il comando passato.

per risolvere il problema che ti da getline , prova a sostituire la funzione , usa gets

Meglio la fgets:

char command[256];
...
fgets(command, sizeof(command), stdin);
command[sizeof(command) - 1] = 0; // forzo il finestringa
...

la gets fa caghèr ():

char command[5];
...
gets(command);
...

prova a digitare da tastiera

asdoufigisevfbspvgdobvuiseh f

tutta sta roba vien messa in una stringa, o meglio, array di 5 elementi...
o segmenta o sporcacci tutta la memoria/variabili memorizzate dopo "command"

[maulattu]

EDIT: doppio...

[DanieleC88]

Originariamente inviato da fero86

tratto da http://linuxmanpages.com/man3/free.3.php :

Ohibò, hai ragione...

[khelidan1980]

Originariamente inviato da fero86

tratto da http://linuxmanpages.com/man3/free.3.php :

Originariamente inviato da DanieleC88

Ohibò, hai ragione...

mah comunque a me su linux dava un errore di compilazione ( o in esecuzione non ricordo bene ) la free su un puntatore null, mentre su window compilava!Me lo ricordo bene perchè ci persi un bel po di tempo

[DanieleC88]

Originariamente inviato da khelidan1980

mah comunque a me su linux dava un errore di compilazione ( o in esecuzione non ricordo bene ) la free su un puntatore null, mentre su window compilava!Me lo ricordo bene perchè ci persi un bel po di tempo

No, mi sa che mi confondevo anche io con una cosa simile (il problema è su una doppia free(), dove non hai resettato il puntatore a NULL ma la memoria è già stata liberata).

[khelidan1980]

Originariamente inviato da DanieleC88

No, mi sa che mi confondevo anche io con una cosa simile (il problema è su una doppia free(), dove non hai resettato il puntatore a NULL ma la memoria è già stata liberata).

già hai ragione era una doppia free!

[-hide-]

Ammazza quante risposte in un giorno
Scusate se io rispondo ora ma sono stato di esami.

Ringrazio per il codice in C++, però è inutile per i miei studi! Io conosco solo il C (anche se noto di riuscire a capirlo un pò).

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

int main() {
	char *psz = NULL;
	size_t cb;
	do {
		free(psz);
		getline(&psz, &cb, stdin);
		system(psz);
	} while (strcmp(psz, "exit"));
	return 0;
}

Ieri mi ero scritto il codice e l'avevo portato all'Uni per dargli un'occhiata nei momenti di noia. Io non comprendo perché deallocare la memoria di una variabile e subito dopo utilizzarla ella getline.
Se non ricordo male, free libera lo spazio di memoria occupato precedentemente dalla variabile psz. La successiva memorizzazione della getline in quella variabile dovrebbe giustamente fallire.

Originariamente inviato da fero86

come sarebbe a dire, non mi dire che non hai mai fatto system("PAUSE") per evitare che la console del programma si chiuda subito

Si certo, ma su quella variabile non so che effetti dia

Originariamente inviato da Teo@Unix

una shell in C (ottimo esercizio secondo me).

Si può fare in vari modi, potresti anche pensare ad un modello client-server... una shell remota insomma forse è ancora meglio come esercizio.

Dovresti darti un occhio al multiplexing in C. Nonchè alla programmazione di rete.

Certo, ma intanto volevo iniziare dal basso. Non ho ancora studiato le chiamate di sistema quindi attenderei per destreggiarmi tra evoluzioni di vario tipo. Grazie!


Ma quindi alla fine, nella sostanza, devo usare la fgets?!

[-hide-]

Facendo due conti ho estrapolato questa versione.

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

int main (void) { // apro main
	char command[256];
	while (strcmp (command, "exit")) { // apro while
		fgets (command, sizeof (command), stdin);
		system (command);
		} // chiudo while
	return 0;
} // chiudo main

- la compilazione viene effettuata (e questo è già un traguardo);
- la comparazione di uscita non la fa, in pratica mi ritrovo in un ciclo infinito;
- funziona solo "ls" sembra

[WarDuck]

A me così funziona:

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

int main(void) {

	char str[256];
	do {
		printf("Cmd> "); // stampa il prompt

		fgets(&str[0], 255, stdin); // acquisisce la stringa dall'input

		system(&str[0]); // esegue il comando specificato in str
	}
	while(!strcmp(&str[0], "exit\n")); // se str contiene "exit" termina

	return 0;
}

strcmp ritorna 0 se sono uguali le stringhe.

[WarDuck]

Versione un po' più "pulita":

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

#define CMD_LENGTH 256

void getLine(char* str, int n) {
	fgets(str, n, stdin);
}

int main(void) {

	char str[CMD_LENGTH];
	do {
		printf("Cmd> ");

		getLine(&str[0], CMD_LENGTH);

		system(str);
	}
	while(strcmp(str, "exit\n"));


	return 0;
}

[-hide-]

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

#define CMD_buffer 256 

int main (void) {
	char cmdline [CMD_buffer];
	while (strcmp (cmdline, "exit\n")) {
		printf ("cmd> ");
		fgets (cmdline, sizeof (cmdline), stdin);
		system (cmdline);
		}
	return 0;
}

Questo codice funziona pure, più o meno!
Non capisco perché "ls" viene riconosciuto mentre "cd" non causa alcun effetto (ne ho scelti due a caso che ho provato).