Discussion :

[tsing]

Bonsoir,

Voilà, j'ai écrit un petit script pour envoyer et télécharger automatiquement un fichier sur un serveur Web mais... lorsque je l'exécute, cette erreur s'affiche :

# sh pti_securite.sh
pti_securite.sh[55]: [: Found: unexpected operator/operand
pfctl: pf already enabled

Naturelement, je vous communique aussi mon code (car sinon vous ne pourriez sans doute pas m'aider :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
#!/bin/sh
# chargement des primitives
 
        fichier_pf_conf="pf.conf"
        adresse_ip_conf="http://192.168.20.2/bts/pti/securite/index.php"
        routeur_nb_conf="1"
 
# dossier courrant
 
        cd /etc/
 
# tests preliminaires
        # existance du fichier
        if [ ! -f $fichier_pf_conf ]
        then
                echo "block all" > $fichier_pf_conf
        fi
 
        # ecriture sur le fichier
        if [ ! -w $fichier_pf_conf ]
        then
                chmod a+w $fichier_pf_conf
        fi
 
# comparaison de la version du fichier
 
        somme_controle=$( md5 $fichier_pf_conf )
        somme_controle=$( echo $somme_controle | cut -c 23-55 )
        reponse=$( lynx -dump $adresse_ip_conf"?numero_routeur="$routeur_nb_conf"&md5="$somme_controle )
 
# comparaison des signatures
 
        #if [ echo $reponse | grep -qF "update" ]
        if [ $reponse = "update" ]
        then
 
        # telechargement de la derniere version du fichier
 
                rm $fichier_pf_conf
                cd /tmp/
                rm *
                wget $adresse_ip_conf"?numero_routeur="$routeur_nb_conf"&download=pf"
                cd /etc/
                mv /tmp/* /etc/$fichier_pf_conf
 
        # chargement du nouveau fichier
 
                pfctl -f $fichier_pf_conf
        fi
 
# activation (ou reactivation) de packet filter
 
        pfctl -e

Donc en gros, je ne sais pas d'où vient mon erreur en fait. Si vous la voyez (ou si vous en avez trouvé plusieurs lol), j'apprécierais bcps que vous puissiez m'en informer : je suis un peu en ...galère lol
Merci pour votre aide, et bonne soirée : codez bien ^_^!!

[gangsoleil]

Bonjour,

1/ As-tu les droits suffisants pour écrire dans le fichier :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

/etc/pf.conf

Car c'est bien ce que tu cherches à faire.

Aprsè, mon shell a du mal avec ca, même si je ne pense pas que l'erreur vienne de là (mais du coup impossible de continuer) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

        somme_controle=$(md5 $fichier_pf_conf)

[tsing]

Chez moi la fonction md5 passe sans problème. Il faut dire qu'OpenBSD (l'OS sur lequel le script s'exécute) contient toute une gamme primitives cryptographique, ce qui facilite le travail.

En fait, moi j'ai un blocage sur l'erreur retournée :

pti_securite.sh[55]: [: Found: unexpected operator/operand

Que signifie [55] ? Et je ne comprends absolument rien à [: Found: unexpected operator/operand aussi... 'fin c'est pas très clair je trouve.

[ggnore]

Dans mes scripts, je ne fais jamais de cd
A quoi ça sert ? on peut mettre des chemins absolus et même stocker ces chemins dans des variables pour rendre la modification ultérieur du script plus facile.

Quand dans mes if [...] j'ai des variables, je les mets systématiquement entre guillemets.

[tsing]

ne fais jamais de cd

Ah bon. Je vais essayer sans. En fait je voulais les utiliser afin de bien cibler les dossiers dans lesquelles je travaille.

[tsing]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if [ $reponse = "update" ]

C'était ça, la boulette
Le test de l'égalité s'écrit comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if [ $reponse -eq "update" ]

Longue vie aux communautés de développement open source.

[ggnore]

ne fais jamais de cd

Ah bon. Je vais essayer sans. En fait je voulais les utiliser afin de bien cibler les dossiers dans lesquelles je travaille.

Tu peux "cibler" tes répertoires sans cd.
C'est typiquement la genre de ligne qui ne sert à rien dans un script.

[tsing]

Tu peux "cibler" tes répertoires sans cd

Absolument. C'est vrai qu'on peut s'en passer, à condition de constamment répéter (parfois lourdement) toute l'arborescence dans laquelle on travaille.

ligne qui ne sert à rien

Peut-être, ou sinon ça évite de cibler constamment une même cible (ça aussi ça "ne sert à rien" ). En tout cas, je trouve que ça rend le script un peu plus lisible, en renseignant sur le répertoire courrant.

[ggnore]

Tu peux "cibler" tes répertoires sans cd

Absolument. C'est vrai qu'on peut s'en passer, à condition de constamment répéter (parfois lourdement) toute l'arborescence dans laquelle on travaille.

ligne qui ne sert à rien

Peut-être, ou sinon ça évite de cibler constamment une même cible (ça aussi ça "ne sert à rien" ). En tout cas, je trouve que ça rend le script un peu plus lisible, en renseignant sur le répertoire courrant.

[...]et même stocker ces chemins dans des variables pour rendre la modification ultérieur du script plus facile[...]

C'est d'expérience que j'ai appris à me méfier/passer des cd.

Si tu regardes des scripts vraiment costauds faits par des pros, tu verras qu'il y a quasi systématiquement 3 tonnes de variables au début de leurs scripts... Dont tous les chemins appelés.
C'est propre, c'est plus facile de maintenir le logiciel, c'est plus souple.

Bonne continuation à toi.

[tsing]

Oui en effet, ça c'est encore mieux. J'y penserai lorsque mes scripts feront plusieurs ko