Erreur de syntaxe près du symbole inattendu « fi »

Question

Bonjour, 

J'ai l'erreur suivante sur mon bash : 

Pouvez vous m'aider ? 

Le bash : 

#!/bin/sh

#Variable carte Ethernet
DEV=vmbr0

#Pour chaque IP dans le fichier list_ip.txt
#Verification si presence dans le fichier Iptable
# - si present, ne rien faire
# - si absent ajout via iptables -I

while read ip; 
		do
			exist=$(iptables -L | grep $ip);

        if [ -z = "$exist" ]; 
		
			then
          #Ajout dans Iptable
          iptables -I FORWARD 1 -o $DEV -s $ip -m limit --limit 2000/sec -j ACCEPT
          iptables -I FORWARD 2 -o $DEV -s $ip -j DROP
        else
          #Deja present pas d ajout dans Iptable
        fi
        
done < list_ip.txt

jisisv · Answer

Remarque ton nom de variable exists n'est pas très heureux (ressemble au mot-clé exit. Utilide plutôt des noms en UPPERCASE_WITH_UNDERSCORES 

Ceci dit,
Pour ton test, ton utilisation est erronée , examine ceci:
johand@bata:~$ unset truc
johand@bata:~$ if [ -z $truc ] ; then echo BAD; fi
BAD
johand@bata:~$ if [ -z = "$truc" ] ; then echo BAD; fi
johand@bata:~$ truc=123
johand@bata:~$ if [ -z $truc ] ; then echo BAD; fi ; echo OK
OK

[ -z = $BLAH ] retourne toujours faux !

En outre, et c'est le plus important, le corps du else est vide ce qui fournit une erreur. Supprime cette branche ou mets-y une instruction valide.
exemple:
#!/bin/sh

if [ -z $1 ] ; then
   echo "No param"
else
# TTT
  echo "Param1 is $1"
fi
Gates gave ^W  sold  you the windows.
GNU gave us the whole house.(Alexandrin)

BashLinux · Answer

Merci de ton retour. 

J'ai update le script, mais j'ai effectivement un retour négatif à chaque fois. A tu une idée stp ? 

#!/bin/sh

# Export IP
#vzlist -H -a -o ip > list_ip.txt

#Variable carte Ethernet
DEV=eth0

#Pour chaque IP dans le fichier list_ip.txt
#Verification si presence dans le fichier Iptable
# - si present, ne rien faire
# - si absent ajout via iptables -L

while read ip;
do
		SI_EXISTE_IPTABLES=$(iptables -L | grep $ip);
        if [ ! -z $SI_EXISTE_IPTABLES ]; then
        #Ajout dans Iptable
        iptables -I FORWARD 1 -o $DEV -s $ip -m limit --limit 100/sec -j ACCEPT
        iptables -I FORWARD 2 -o $DEV -s $ip -j DROP
		echo "IP Ajoute"
else
        echo "$ip deja existante"
fi
done < list_ip.txt


Merci du retour :)

jisisv · Answer

iptables est-il dans ton PATH ?
exemple sous ma Debian :
johand@bata:~$ which iptables
johand@bata:~$ ls -l /sbin/iptables
lrwxrwxrwx 1 root root 13 nov  8  2014 /sbin/iptables -> xtables-multi
 
Vérifie l'utilisateur de ton script et/ou utilise sudo ou toute mesure appropriée.

Erreur de syntaxe près du symbole inattendu « fi »

3 réponses

Votre réponse

Discussions similaires