sh / cygwin probleempje

[Saven]

Hallo,
ik moet even snel iets testen voor mijn server waar ik geen ssh toegang tot heb, dus gebruik ik cygwin voor op mijn eigen pc.

Alleen tijdens het uitvoeren geeft hij een fout:


Ik heb uren zitten kijken naar dit stukje code:

Bash:

 e=0; l=0; echo "$HTTP_BACK" | while read i; do let l++;
   if [ $l = 1 ] && [ "$i" != "$HTTPOKMSG$RETURNCHR" ] \
&& [ "$i" != "$HTTP1KMSG$RETURNCHR" ]; then
      echo "Bad Server Response :\\"; exit 4; fi;

Het gaat dus om de 2 middelste regels, maar ik kom er echt niet uit. Kan iemand mij verlossen uit deze rare error? Thnx

edit: voor de duidelijkheid, ik gebruik dus Cygwin, voor Windows dus

[ Voor 5% gewijzigd door Saven op 15-02-2009 23:30 ]

[NMe]

Waar hoort mijn topic?

PRG>>NOS

[Saven]

Sorry het is Windows

[blaataaps]

Wat heb je al geprobeerd? Wat is de bron van het script, wat moet het precies doen? Welke shell gebruikt het? Wat gebeurt er als het op 1 regel staat? Ligt het aan mij of staat er nergens done?

[Saven]

Ik heb dit script van internet om de veiligheid van mijn server te testen. 'Done' ben ik nergens tegen gekomen, maar ik heb dan ook de ballen verstand van shell scripting.

Heb even zitten zoeken waarom done nodig zou zijn, maar het script gebruikt geen loops

[jelly]

Saven schreef op maandag 16 februari 2009 @ 13:52:
Ik heb dit script van internet om de veiligheid van mijn server te testen. 'Done' ben ik nergens tegen gekomen, maar ik heb dan ook de ballen verstand van shell scripting.

Heb even zitten zoeken waarom done nodig zou zijn, maar het script gebruikt geen loops

Als je de veiligheid wil testen van een server met een script ben je volgens mij niet helemaal handig bezig, er zijn gewoon programma zoals nessus waarmee je server porten etc kan scannen. Lijkt mij wat veiliger dat zomar een script van iemand kopieren, het script kan net zogoed wget $virus bevatten

[MBV]

Saven schreef op maandag 16 februari 2009 @ 13:52:
Ik heb dit script van internet om de veiligheid van mijn server te testen. 'Done' ben ik nergens tegen gekomen, maar ik heb dan ook de ballen verstand van shell scripting.

Heb even zitten zoeken waarom done nodig zou zijn, maar het script gebruikt geen loops

Dan zou ik dit script maar nooit gebruiken, want kennelijk heb je geen idee wat het script doet.

Wat denk je dat hier gebeurt:

while read i; do let l++;

[Kees]

MBV schreef op maandag 16 februari 2009 @ 14:59:
[...]

Dan zou ik dit script maar nooit gebruiken, want kennelijk heb je geen idee wat het script doet.

Wat denk je dat hier gebeurt:

while read i; do let l++;

hij leest een response (schijnbaar van een http verbinding) en voor elke regel die hij krijgt verhoogt hij die variable 'l' met 1 - wat denk jij dat er gebeurd?

Wat je zou kunnen doen met de code is het opschonen en voorzien van wat identatie; alla:

ee=0
l=0
echo "$HTTP_BACK" | while read i 
do 
	let l++;
	if [ $l = 1 ] && [ "$i" != "$HTTPOKMSG$RETURNCHR" ] && [ "$i" != "$HTTP1KMSG$RETURNCHR" ]
	then
		echo "Bad Server Response :\\"
		exit 4
	fi
[rest van het script]

[ Voor 29% gewijzigd door Kees op 16-02-2009 15:34 ]

[leuk_he]

Saven schreef op zondag 15 februari 2009 @ 23:23:

Bash:

 e=0; l=0; echo "$HTTP_BACK" | while read i; do let l++;
   if [ $l = 1 ] && [ "$i" != "$HTTPOKMSG$RETURNCHR" ] \
&& [ "$i" != "$HTTP1KMSG$RETURNCHR" ]; then
      echo "Bad Server Response :\\"; exit 4; fi;

ik zou het zo schrijven... wellicht ook verschil tussen ksh/bash/zsh enzo...

code:

   if  [  $l = 1 -a  "$i" != "$HTTPOKMSG$RETURNCHR" -a "$i" != "$HTTP1KMSG$RETURNCHR" ]
 then
      echo "Bad Server Response :\\"; exit 4; 
 fi;

Na de eerste "]" moet een then. ( man test is manual pagina voor [ ] )

[ Voor 7% gewijzigd door leuk_he op 16-02-2009 15:43 . Reden: newlines toegevoegd ]

[Kees]

leuk_he schreef op maandag 16 februari 2009 @ 15:31:
[...]


ik zou het zo schrijven... wellicht ook verschil tussen ksh/bash/zsh enzo...

code:

   if  [  $l = 1 -a  "$i" != "$HTTPOKMSG$RETURNCHR" -a "$i" != "$HTTP1KMSG$RETURNCHR" ] then
      echo "Bad Server Response :\\"; exit 4; fi;

Na de eerste "]" moet een then. ( man test is manual pagina voor [ ] )

bij bash iig moet de 'then' op de volgende regel staan, of na een ';' komen

[leuk_he]

Kees schreef op maandag 16 februari 2009 @ 15:33:
[...]

bij bash iig moet de 'then' op de volgende regel staan, of na een ';' komen

Mijn leermoment van vandaag.. Ik heb het hierboven niet gedaan, maar zet het inderdaad altijd op de volgende regel.

[MBV]

Kees schreef op maandag 16 februari 2009 @ 15:26:
[...]

hij leest een response (schijnbaar van een http verbinding) en voor elke regel die hij krijgt verhoogt hij die variable 'l' met 1 - wat denk jij dat er gebeurd?

Ik had geen zin om me erin te verdiepen, maar het viel me nogal op dat de TS stellig beweert dat er geen loops worden gebruikt, en dat er vervolgens wel een while-loop staat. Daarom zei ik dat hij het script maar beter niet kon gebruiken, als je niet snapt wat een willekeurig script van het internet doet kan je het IMHO maar beter niet vertrouwen.

[sam.vimes]

Wat een onleesbare lay-out. Vervangen van ';' door newlines en toevoegen van indentatie geeft:

Bash:

e=0
l=0
echo "$HTTP_BACK" |
while read i
do
    let l++
    if [ $l = 1 ] && [ "$i" != "$HTTPOKMSG$RETURNCHR" ] \
        && [ "$i" != "$HTTP1KMSG$RETURNCHR" ]
    then
        echo "Bad Server Response :\\"
        exit 4
    fi

Er ontbreekt een afsluitende 'done' bij de 'while do', maar dat komt denk ik omdat het een fragment is.
Verder is de syntax op het oog correct.
Conclusie: er is iets onzichtbaars gebeurd.
Ik denk dat het script DOS regeleinden heeft gekregen of dat er een spatie achter de backslash van de 'if'-regel terecht is gekomen.
Oplossing: converteer naar unix regeleinden (enkel LF, geen CRLF)
of: join de 'if' regel en de '&&' regel tot 1 regel en verwijder de backslash:

Bash:

    if [ $l = 1 ] && [ "$i" != "$HTTPOKMSG$RETURNCHR" ]  && [ "$i" != "$HTTP1KMSG$RETURNCHR" ]
    then

[zomertje]

Van dat soort zaken heb ik zowel op Unix als op Windows last gehad: onzichtbare tekens die je code niet laten werken. Kan dus een goeie optie zijn.