From: Heiko Hund <he...@ist.eigentlich.net>
In the resolvconf part of the script there was one instance of a dynamic
variable using _* left. The _* ones do not work as the regular ones, but
only when you directly place them within ${!}, not indirectly using a
variable.
Convert the code to use a loop and a check, like in all the other places
in the script.
Change-Id: Id800cad0e92e0abc0d96079fdb5a9d57578e1446
Signed-off-by: Heiko Hund <he...@ist.eigentlich.net>
Acked-by: Frank Lichtenheld <fr...@lichtenheld.com>
---
This change was reviewed on Gerrit and approved by at least one
developer. I request to merge it to master.
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1144
This mail reflects revision 1 of this Change.
Acked-by according to Gerrit (reflected above):
Frank Lichtenheld <fr...@lichtenheld.com>
diff --git a/distro/dns-scripts/systemd-dns-updown.sh
b/distro/dns-scripts/systemd-dns-updown.sh
index 9006e28..ed3947a 100644
--- a/distro/dns-scripts/systemd-dns-updown.sh
+++ b/distro/dns-scripts/systemd-dns-updown.sh
@@ -189,11 +189,13 @@
domains+="${!domain_var} "
done
{
+ local i=1
local maxns=3
- local server_var=dns_server_${n}_address_*
- for addr_var in ${!server_var}; do
- [ $((maxns--)) -gt 0 ] || break
+ while [ "${i}" -le "${maxns}" ]; do
+ local addr_var=dns_server_${n}_address_${i}
+ [ -n "${!addr_var}" ] || break
echo "nameserver ${!addr_var}"
+ i=$((i+1))
done
[ -z "$domains" ] || echo "search $domains"
} | /sbin/resolvconf -a "$dev"
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
