From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: [RFC PATCH 4/8] network-hotplug-bridges: Fix warning when address
 doesn't exist
Date: Thu, 02 Dec 2021 12:37:44 +0000
Message-ID: <20211202123748.2525617-4-michael.tremer@ipfire.org>
In-Reply-To: <20211202123748.2525617-1-michael.tremer@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8974405059223695832=="
List-Id: <development.lists.ipfire.org>

--===============8974405059223695832==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Some devices do not have an address, which prints an error when reading
it.

Signed-off-by: Michael Tremer <michael.tremer(a)ipfire.org>
---
 config/udev/network-hotplug-bridges | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/config/udev/network-hotplug-bridges b/config/udev/network-hotplu=
g-bridges
index bacf722ef..4b8764598 100644
--- a/config/udev/network-hotplug-bridges
+++ b/config/udev/network-hotplug-bridges
@@ -41,9 +41,8 @@ detect_zone() {
 		# Try to find out if this INTERFACE is a slave of a zone
 		local slave
 		for slave in $(get_value "${zone}_SLAVES"); do
-
-			#Compare if the mac address matches or if the name matches
-			if ([ "$(</sys/class/net/${INTERFACE}/address)" =3D "${slave}" ] || [ "${=
INTERFACE}" =3D "${slave}" ]); then
+			# Compare if the mac address matches or if the name matches
+			if [ -r "/sys/class/net/${INTERFACE}/address" -a "$(</sys/class/net/${INT=
ERFACE}/address)" =3D "${slave}" ] || [ "${INTERFACE}" =3D "${slave}" ]; then
 				echo "${zone}"
 				return 0
 			fi
--=20
2.30.2


--===============8974405059223695832==--