Project

General

Profile

Anomalie #7444

Erreur de variable chaine_T1 => chaine_T2

Added by Christophe LEON about 7 years ago. Updated almost 7 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Category:
-
Start date:
Due date:
03/07/2014
% Done:

100%

Estimated time:
1.00 h
Spent time:
Distribution:
EOLE 2.3

Description

Ligne 240: la variable ne devrait pas être chaine_T1 mais chaine_T2

agregation.sh.patch View (487 Bytes) Karim Ayari, 02/19/2014 04:32 PM

agregation.sh.patch View (980 Bytes) Karim Ayari, 02/26/2014 12:10 PM

Associated revisions

Revision ee62c721 (diff)
Added by Joël Cuissinat about 7 years ago

agregation.sh : correction chaine_T1 => chaine_T2

Application du patch correctif agregation.sh.patch
Thanks: Karim Ayari
Fixes: #7444 @20m

Revision da6b7261 (diff)
Added by Joël Cuissinat about 7 years ago

agregation.sh : correction des autres chaînes

Application du patch correctif agregation.sh.patch (n°2)
Thanks: Karim Ayari
Thanks: Christophe Léon
Fixes: #7444 @10m

History

#1 Updated by Christophe LEON about 7 years ago

Dans le même ordre d'idée; je trouve étrange les commandes suivantes

# Vidage des chaines MANGLE
check_T1=$(iptables-save |grep "RESTOREMARK" |wc -l)
check_T2=$(iptables-save |grep "T2" | wc -l)
check_RESTOREMARK=$(iptables-save |grep "T1" |wc -l)
check_PREROUTING=$(iptables-save |grep "T1" |wc -l)

J'aurais plutôt vu,

check_T1=$(iptables-save |grep "T1" |wc -l)
check_T2=$(iptables-save |grep "T2" | wc -l)
check_RESTOREMARK=$(iptables-save |grep "RESTOREMARK" |wc -l)
check_PREROUTING=$(iptables-save |grep "PREROUTING" |wc -l)

mais je me trompe peut être :)

#2 Updated by Karim Ayari about 7 years ago

bien vu :) je devais être amoureux ce jour là ^^

ok pour correction remplacer une des lignes

check_PREROUTING=$(iptables-save |grep "T1" |wc -l)

par

check_PREROUTING=$(iptables-save |grep "PREROUTING" |wc -l)

#3 Updated by Joël Cuissinat about 7 years ago

  • Assigned To set to Joël Cuissinat
  • Target version set to Mises à jour 2.3.13
  • Estimated time set to 1.00 h

#4 Updated by Karim Ayari about 7 years ago

  • File agregation.sh.patch View added
  • Assigned To deleted (Joël Cuissinat)
  • Target version deleted (Mises à jour 2.3.13)

voilà le patch

#5 Updated by Joël Cuissinat about 7 years ago

  • Due date set to 03/07/2014
  • Assigned To set to Joël Cuissinat
  • Target version set to Mises à jour 2.3.13

#6 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#7 Updated by Christophe LEON about 7 years ago

Merci pour la prise en compte de ce signalement, cependant voici d'une manière plus précise l'objet de cette demande

http://dev-eole.ac-dijon.fr/projects/conf-amon/repository/revisions/2.3/entry/bin/agregation.sh

Ligne 240: il y a

if [ -z "$chaine_T1" ] ; then

C'est la même ligne que 230, je pense plutôt que c'est if [ -z "$chaine_T2" ] ; then

Ligne 211: il y a

check_T1=$(iptables-save |grep "RESTOREMARK" |wc -l)

et ligne 213: il y a

check_RESTOREMARK=$(iptables-save |grep "T1" |wc -l)

Vu que check_T1 est utilisé plus bas pour vider les regles de la table T1 et
Vu que check_RESTOREMARK est utilisé plus bas pour vider les regles RESTOREMARK

J'aurais pour plus de cohérence modifié comme suit

ligne 211 check_T1=$(iptables-save |grep "T1" |wc -l)
ligne 213 check_RESTOREMARK=$(iptables-save |grep "RESTOREMARK" |wc -l)

j'aurais aussi une modification a apporter dans le /etc/init.d/agregation, je ferais un autre signalement :)

#8 Updated by Karim Ayari about 7 years ago

oui effectivement j'avais pas compris et encore moins ouvert grand mes yeux. je prépare un patch pour toutes ces modifs.

#9 Updated by Karim Ayari about 7 years ago

un patch rassemblant toutes les modifications.

#10 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Résolu to Réouvert
  • % Done changed from 100 to 50

#11 Updated by Joël Cuissinat about 7 years ago

  • Status changed from Réouvert to Résolu
  • % Done changed from 50 to 100

#12 Updated by Joël Cuissinat about 7 years ago

Paquet candidat 2.3 : conf-amon >= 2.3-eole103~1
Paquet unstable 2.4 : conf-amon >= TODO

#13 Updated by Klaas TJEBBES almost 7 years ago

  • Status changed from Résolu to Fermé

Confirmé par Karim.

Also available in: Atom PDF