[PATCH] Fixed unwanted reboot every 5 Minutes after -n flash

Tim Niemeyer tim at tn-x.org
Fr Jan 6 12:41:15 CET 2017


Moin

Am Dienstag, den 27.12.2016, 13:38 +0100 schrieb Jan Kraus:
> The error only occurs if there isn't any network connection.
> Closes https://mantis.freifunk-franken.de/view.php?id=23
Hier wäre es schön "Fixes #23" zu schreiben. Das kann das Mantis
inzwischen auch erkennen und dann die Tickets automatisch auf resolved
setzen.

> 
> Signed-off-by: Jan Kraus <mayosemmel at gmail.com>
Schaut gut aus, wobei man vermutlich ein paar Zeilen drunter nochmal
zusätzlich prüfen sollte, ob wirklich Koordinaten gesetzt sind oder
nicht.

------ %< ------
 hostname=$( echo $xml_data |grep -o  '<hostname>.*<\/hostname>'|sed -e 's/<\/\?hostname>//g' )
 description=$( echo $xml_data |grep -o  '<description>.*<\/description>'|sed -e 's/<\/\?description>//g' )
 latitude=$( echo $xml_data |grep -o  '<latitude>.*<\/latitude>'|sed -e 's/<\/\?latitude>//g' )
 longitude=$( echo $xml_data |grep -o  '<longitude>.*<\/longitude>'|sed -e 's/<\/\?longitude>//g' )
+
+if [ -z "$latitude" ] || [ -z "$longitude" ]; then
+    echo "No coordinates, looks like an error"
+    exit 1
+fi
 
 #Check for netmon default coordinates
 if [ "$latitude" -eq 0 ] && [ "$longitude" -eq 0 ]; then
     echo "Unable to retrieve coordinates from Netmon."
------ >% ------

Ist nur ein Vorschlag. Ich würde den Commit auch so nehmen, von daher:
Reviewed-by: Tim Niemeyer <tim at tn-x.org>

Tim

> ---
>  .../fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh       | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh b/src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh
> index 4c20974..1448014 100755
> --- a/src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh
> +++ b/src/packages/fff/fff-sysupgrade/files/usr/sbin/copy_netmon_coordinates.sh
> @@ -26,6 +26,9 @@ xml_data=$( wget -q -O - "http://fe80::ff:feee:1%br-mesh/api/rest/api.php?rquest
>  if [ -z "$xml_data" ]; then
>  	echo "xml_data is not set"
>  	exit 1
> +elif echo "$xml_data"|grep "can't connect to remote host" ;then
> +	echo "The Netmon Server cannot be reached."
> +	exit 1
>  elif echo "$xml_data"|grep "HTTP/1.1 404 Not Found" ;then
>  	echo "This Router is not present in the Netmon Database."
>  	delete_myself
> -- 
> 2.1.4
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20170106/ebe723ec/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev