ltuffery / minishell Goto Github PK

View Code? Open in Web Editor NEW

2.0 2.0 0.0 296 KB

mineshoul

Makefile 2.58% C 95.88% Shell 1.54%

minishell's Introduction

👋 Hey! I'm ltuffery

Student at ecole 42

Do you want to know the strange history of Segfault? Well, you won't know any more than I do

minishell's People

Contributors

Stargazers

Watchers

minishell's Issues

Probleme de Ctrl-C sur la cmd cat

quand nous faisons cat puis Ctrl-C cela casse le minishell et donne un resultat etrange, ceci est probablement du au signaux qui ne sont pas refait dans le fork

decoupe de commande

Tockens_manager ajoute une entré si il y a des qutes en dernier caractère se qui ajoute une commande fantome.
<<"limiter" -> une commande famtome apparait et execute une commande vide au lieux de ne rien faire

get_cmd -> le split de la commande

la fonction get_cmd() qui permet de split en un tableau de char* les commandes entre pipes (en tenant compte des quotes), ne fonctionne pas parfaitement, EX: echo "coucou vous" >out
renvoi :
cmdx[0] -> echo
cmdx[1] -> coucou vous
cmdx[2] -> non
cmdx[3] -> charabia
Mais echo "coucou vous ou cmd >out >out <in Fonctionne correctement

Manque dans la gestion des var d'env

les values des var d'env doivent être split en plusieur argument à chaque espace (redirection et commande) sauf si la variable est entre double quote "$VAR" la elle ne doit pas être split

compile pas

ça marche pas

export erreur

Si nous faisons : export toto="a b" puis export toto cela ecrase l'ancienne variable alors qu'il ne devrais pas car toto contient quelque chose

Probleme d'exitcode sur les builtins

la plus part des builtins ne sortent pas le bon exitcode quand ils ont une erreur

Problème parsing chevron

Les chevrons <> revoie une erreur mais <>out fonctionne

<<< ne marche pas non plus sur minishoul

Nom de redirection n'est pas bon

quand nous faisons : echo oui >$USER" o" cela cree un fichier ltuffery et pas ltuffery o

Sapce cmd

Quand nous ecrivons :

"     " coucou

cela doit dire commande " " not found
Mais le code actuel skit la cmd espace et prend coucou comme cmd....

Problème lors de pipe/commande fausse.

lors de commandes combinées avec des pipe contenant des commande fausses, on as des erreurs.
Ex : <infile cat | cat | invalid_cmd | cat
4 fd reste ouverte (pipe non fermé)
1 leak provenant de val_value

==128659==    by 0x402BC5: var_value (interpreter_var.c:76)
==128659==    by 0x40324B: get_replace_var (get_cmd.c:130)
==128659==    by 0x4035B1: get_cmd (get_cmd.c:176)
==128659==    by 0x402438: add_cmd (split_prompt.c:27)
==128659==    by 0x4023A6: split_pipe (split_prompt.c:54)
==128659==    by 0x402305: cut_prompt (split_prompt.c:82)
==128659==    by 0x4021B9: lexer_handler (lexer.c:20)
==128659==    by 0x401A29: input_handler (minishell.c:88)
==128659==    by 0x4019E0: call_promt (minishell.c:112)
==128659==    by 0x40181C: main (minishell.c:164)

ctrl-c probleme

Quand nous commencons a ecrire une commande puis nous coupons avec crtl-c il n'est pas possible d'executer crtl-D sur la nouvelle ligne crée

exit code avec agument (+ et -)

exit 12 ou ```exit -12````

free des fork

Revoir les free des fork, tout n'est pas free en cas d'erreur execve (single et probablement pipe)
ET les closes de fichier dans les fork !?

Variable d'env

La variable d'environement doit renvoyer le '$' si une erreur dans l'ecriture de la var (si apres le '$' il y a autre chose qu'un alpha et '_' (pas de num))

Error parsing

Le parsing sur la redirection me fait des faux positif sur les pipe:
<<h >out | <<h >out2 me renvoie -> minishoul: syntax error near unexpected token '|'

exit avec parametre

exit -1
exit more_long_long
exit doit fonctionner avec des nombres négatifs
exit doit renvoyer une erreur lorsque le nombre dépasse un long long int (et exit ensuite)

Probleme lexing file

Mauvaise récupération de file lors d'un exemple comme suit :
echo coucou >out<<heredoc
Le programme le récupère comme suit :
File1 -> >out<<heredoc
File2 -> <<heredoc
au lieux de :
File1 -> >out
File2 -> <<heredoc

Ctrl-C duplique ligne

Qund nous commemcons a taper une ligner puis que nous faisons Ctrl-C cela duplique la ligne
Exemple :

minishoul> test^C
minishoul> test

Le variable avec espace ou tab n'ont pas le bon comportement

export var=" o " ne doit pas mettre les espace quand il n'est pas entre quote

Non prise en charge des sorties de redirection dans le lexer

ls > dirlist 2>&1

la 2e redirection ne part pas dans le sortie 2 donc cela donne 2 redirection avec comme nom respectif : dirlist et &1 et ls prend 2 comme arg.
Bash ne fais pas du tout ça

Invalite exitcode

Quand nous faisons cat | ls et Ctrl-C le cat notre $? vaut 130 alors qu'il devrais etre 0

code erreur builtins (cd)

en cas d'erreur cd doit retourner 1
cd many argument -> 1
valable pour cd et peut-être d'autres builtins

Valgrind gueule sur le "get_cmd" (logique)

Quand je tape :

echo $USER""Oui

Valgrind renvoie une erreur invalide read pour le reste car le strjoin de l'insertion du contenue de la variable racourci la taille de tmp, donc il vat falloir réalouer char par char sauf si tu as une meilleurs idée

bug redirection

bug lors d'une redirection vers variable d'environnement existante mais vide.
export toto cat <$toto

Probleme de free

quand nous faisons :

> export toto="   a   "
> echo$toto"oui"
> export toto=a
> echo$toto"oui"

il y a une perte de memoire

pipe tout casser

quand nous faisons : <builtins|cmd not found> | ls nous avons cette erreur :

/usr/bin/ls: No such file or directory

cela marche que pour les commande qui passe dans le /usr/bin

Manque parsing

Il manque du parsing pour la redirection input/output file. (< et >).
par exemple :
> > outfile - > syntax error near unexpected token '>'
echo coucou > >out | echo nice -> aucune des deux commande n'est exécuté, aucun fichier créer. -> Parsing.
Même soucis pour le infile < < infile cat -e -> syntax error near unexpected token '>'
Egalement il faut gérer qu'il n'y ai rien apres cat >

Premiere execution d'une commade

==81095== Conditional jump or move depends on uninitialised value(s)                                                                                                                                                                          
==81095==    at 0x4928958: __vfprintf_internal (vfprintf-internal.c:1687)                                                                                                                                                                     
==81095==    by 0x4912D3E: printf (printf.c:33)                                                                                                                                                                                               
==81095==    by 0x4012B4: print_struct (minishell.c:27)                                                               
==81095==    by 0x401683: input_handler (minishell.c:60)                                                              
==81095==    by 0x40162A: call_promt (minishell.c:73)                                                                 
==81095==    by 0x4015BD: main (minishell.c:113)                                                                      
==81095==                                                                                                             
==81095== Use of uninitialised value of size 8         
==81095==    at 0x490C69B: _itoa_word (_itoa.c:179)     
==81095==    by 0x4928574: __vfprintf_internal (vfprintf-internal.c:1687)                                              
==81095==    by 0x4912D3E: printf (printf.c:33) 
==81095==    by 0x4012B4: print_struct (minishell.c:27)
==81095==    by 0x401683: input_handler (minishell.c:60)
==81095==    by 0x40162A: call_promt (minishell.c:73)
==81095==    by 0x4015BD: main (minishell.c:113)                                                                      
==81095==                                                  
==81095== Conditional jump or move depends on uninitialised value(s)                                                   
==81095==    at 0x490C6AD: _itoa_word (_itoa.c:179)     
==81095==    by 0x4928574: __vfprintf_internal (vfprintf-internal.c:1687)                                              
==81095==    by 0x4912D3E: printf (printf.c:33) 
==81095==    by 0x4012B4: print_struct (minishell.c:27)
==81095==    by 0x401683: input_handler (minishell.c:60)                                                              
==81095==    by 0x40162A: call_promt (minishell.c:73)
==81095==    by 0x4015BD: main (minishell.c:113)                                                                      
==81095==                                                  
==81095== Conditional jump or move depends on uninitialised value(s)                                                   
==81095==    at 0x4929228: __vfprintf_internal (vfprintf-internal.c:1687)                                              
==81095==    by 0x4912D3E: printf (printf.c:33)      
==81095==    by 0x4012B4: print_struct (minishell.c:27)
==81095==    by 0x401683: input_handler (minishell.c:60)
==81095==    by 0x40162A: call_promt (minishell.c:73)                                                                 
==81095==    by 0x4015BD: main (minishell.c:113)                                                                      
==81095==                                                  
==81095== Conditional jump or move depends on uninitialised value(s)                                                   
==81095==    at 0x49286EE: __vfprintf_internal (vfprintf-internal.c:1687)                                              
==81095==    by 0x4912D3E: printf (printf.c:33)      
==81095==    by 0x4012B4: print_struct (minishell.c:27)
==81095==    by 0x401683: input_handler (minishell.c:60)
==81095==    by 0x40162A: call_promt (minishell.c:73)                                                                 
==81095==    by 0x4015BD: main (minishell.c:113)                                                                      
==81095==