Jump to content

probleme d'expr en script shell


Recommended Posts

Salut à tous, SVP j'ai un petit probleme avec ce script, il ne veut pas tourner, merci de bien vouloir m'aider:

#!/bin/sh

if [ -f ram ]

then

rm ram;

fi

if [ -f ram2 ]

then

rm ram2;

fi

grep "MemTotal" /proc/meminfo > ram2;

grep "MemFree" /proc/meminfo >> ram2;

grep "Buffers" /proc/meminfo >> ram2;

sed "s/[ ][ ]*/:/g" ram2 > ram;

sed "s/[:][:]*/:/g" ram > ram2;

memTotal=`grep "MemTotal" ram2 | cut -d: -f2 ram2`;

memFree=`grep "MemFree" ram2 | cut -d: -f2 ram2`;

memBuffers=`grep "Buffers" ram2 | cut -d: -f2 ram2`;

memUsed=`expr $memTotal - $memFree`;

echo "Ram: $memTotal :Total: $memUsed :Used: $memFree :Free: $Buffers :Buffers" > ram;

more ram;

et ce en m'indiaquant:

linux:/home/HacH # sh ram.sh

expr: erreur de syntaxe

Ram: 515076

13792

55340 :Total: :Used: 515076

13792

55340 :Free: :Buffers

Merci bien de m'aider...

PS: desolé je devrait essayer de mettre des capture au lieu ce code mais ca n'a pas voulu marcher.

Link to comment
Share on other sites

memUsed=`expr $memTotal - $memFree`;

memUsed=$((expr $memTotal - $memFree))

D'ailleurs toujours remplacer les `` par des $() en bash (pour les scripts sh uniquement, il faut s'en tenir aux `).

PS: desolé je devrait essayer de mettre des capture au lieu ce code mais ca n'a pas voulu marcher.
Non surtout pas .

EDIT :

Tiens pendant que j'y pense :

grep "MemTotal" /proc/meminfo > ram2

grep "MemFree" /proc/meminfo >> ram2

grep "Buffers" /proc/meminfo >> ram2

egrep "MemTotal|MemFree|Buffers" /proc/meminfo

Plus propre et plus lisible

EDIT 2: moi j'aurais fait comme ça :

for pattern in MemTotal MemFree Buffers; do 
 echo -e "$pattern :"
 grep $pattern /proc/meminfo | awk {'print $2'}
done

Link to comment
Share on other sites

D'ailleurs toujours remplacer les `` par des $() en bash (pour les scripts sh uniquement, il faut s'en tenir aux `).

Avec le sh livré avec bash ça marche aussi, les $( )

memUsed=$((expr $memTotal - $memFree))

Euh non ça ne marche pas ça...

Là je vois 3 manières de le faire :

  • memUsed=`expr $memTotal - $memFree`
  • memUsed=$(expr $memTotal - $memFree)
  • memUsed=$(($memTotal - $memFree))

EDIT : pour les warriors :

#!/bin/sh
eval $(sed -n -r -e 's/^(MemTotal|MemFree|Buffers):\s*([0-9]*).*/\1=\2/g p' /proc/meminfo)
echo "Ram: Total:$MemTotal Used:$(($MemTotal-$MemFree)) Free:$MemFree Buffers:$Buffers"

Link to comment
Share on other sites

Avec le sh livré avec bash ça marche aussi, les $( )

Si c'est un lien symbolique vers bash (ou dash chez debian), c'est sûr :chinois:

Mais avec un vrai sh, ça ne fonctionne pas.

memUsed=$((expr $memTotal - $memFree))

Euh non ça ne marche pas ça...

Oui, bien évidemment, il ne faut pas tenir compte du expr qui est resté par erreur.
#!/bin/sh
eval $(sed -n -r -e 's/^(MemTotal|MemFree|Buffers):\s*([0-9]*).*/\1=\2/g p' /proc/meminfo)
echo "Ram: Total:$MemTotal Used:$(($MemTotal-$MemFree)) Free:$MemFree Buffers:$Buffers"

Jolie solution.
Link to comment
Share on other sites

#!/bin/sh
eval $(sed -n -r -e 's/^(MemTotal|MemFree|Buffers):\s*([0-9]*).*/\1=\2/g p' /proc/meminfo)
echo "Ram: Total:$MemTotal Used:$(($MemTotal-$MemFree)) Free:$MemFree Buffers:$Buffers"

Jolie solution.

je ne sais pas si on peux qualifier une solution en -wx--x--x de jolie... :chinois:

Link to comment
Share on other sites

d'abord je vous remercie les gars pour vos apports malgré que j'ai pas encore tester faite de temps, mais bref SVP:

for pattern in MemTotal MemFree Buffers; do

echo -e "$pattern :"

grep $pattern /proc/meminfo | awk {'print $2'}

done

que fait exactement awk dans ca cas.

je ne sais pas si on peux qualifier une solution en -wx--x--x de jolie...
??????

pourquoi?

Mer6....

Link to comment
Share on other sites

--wx--w--w

c'est le résultat d'un chmod a-r sur un shell normal (en -rwx-rw-rw-)

chmod a-r signifiant : enlever l'attribut lisible pour le fichier ... :chinois: lorinc pour l'astuce :D

sinon on peut aussi utiliser l'output de vmstat, hein :francais:

Link to comment
Share on other sites

d'abord je vous remercie les gars pour vos apports malgré que j'ai pas encore tester faite de temps, mais bref SVP:

for pattern in MemTotal MemFree Buffers; do

echo -e "$pattern :"

grep $pattern /proc/meminfo | awk {'print $2'}

done

que fait exactement awk dans ca cas.

je ne sais pas si on peux qualifier une solution en -wx--x--x de jolie...
??????

pourquoi?

Mer6....

awk affiche le deuxième argument qui lui a été passé, et comme avec avec le pipe tu lui passes un truc du style "MemTotal 65535 Mo", et bien le deuxième argument est 65535 :chinois:

Link to comment
Share on other sites

Là je vois 3 manières de le faire :
  • memUsed=`expr $memTotal - $memFree`
  • memUsed=$(expr $memTotal - $memFree)
  • memUsed=$(($memTotal - $memFree))

En fait il y a aussi :
let memused = "$memTotal - $memFree"
((memused=memTotal-memFree))

Et pour info attention avec eval. Ne *jamais* utiliser eval avec des variables dont on ne contrôle pas l'entrée !

Link to comment
Share on other sites

La grippe aviaire.

Plus sérieusement suppose un eval $ma_var qui est remplie par un utilisateur.

Ton utilisateur un peu malicieux va mettre dans ma_var une chaine du genre "echo coucou;rm -rf ~" (en supposant que le suid soit sur le script, sinon le home du propriétaire en dur).

Résultat en remplaçant, ça va faire

eval echo coucou;rm -rf ~

Le eval s'arrête au ; et le reste est traité en tant que shell script.

En fait le nom eval vient d'une erreur. L'inventeur d'eval voulait en fait l'appeller evil, mais il a fait une faute de frappe et c'est resté eval.

Link to comment
Share on other sites

Plus sérieusement suppose un eval $ma_var qui est remplie par un utilisateur.

Ton utilisateur un peu malicieux va mettre dans ma_var une chaine du genre "echo coucou;rm -rf ~" (en supposant que le suid soit sur le script, sinon le home du propriétaire en dur).

Résultat en remplaçant, ça va faire

eval echo coucou;rm -rf ~

Le eval s'arrête au ; et le reste est traité en tant que shell script.

Même pas... si tu met ma_var="rm -rf ~" ça marche aussi (c'est même fait exprès)...

(bon, en passant par sed avant et vu la regex utilisée je pense qu'il n'y a pas trop de risque :chinois: )

Link to comment
Share on other sites

Ouais c'est vrai en plus.

Je pensait à un exemple pratique plus compliqué.

Bon pour ici c'est pas trop grave, c'est un fichier qu'on peut truster quand même, c'est pour ça que j'ai précisé une entrée qu'on ne contrôle pas : une variable en paramètre ou un/des fichier (touch "rm -rf *" ou un truc dans le genre).

Link to comment
Share on other sites

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...