From abc8952153f7da46744b0bfe28cf91b3fad567b6 Mon Sep 17 00:00:00 2001
From: bergware <bergware@users.noreply.github.com>
Date: Wed, 13 Dec 2023 13:45:01 +0100
Subject: [PATCH] Docker / VM: avoid unnecessary writes to flash device

---
 etc/rc.d/rc.libvirt | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/etc/rc.d/rc.libvirt b/etc/rc.d/rc.libvirt
index 82ec46c4b..6a7b3d8de 100755
--- a/etc/rc.d/rc.libvirt
+++ b/etc/rc.d/rc.libvirt
@@ -207,13 +207,17 @@ libvirtd_start(){
       if [[ ! -e $SYSTEM/$DEV ]]; then
         NAME=${DEV//[0-9.]/}
         if [[ $NAME == br ]]; then
-          # change to macvtap
-          log "change $DEV to macvtap in $XML"
-          sed -ri "${ROW[$i]} s/<interface type='bridge'>/<interface type='direct' trustGuestRxFilters='yes'>/; $ROW2 s/<source bridge='$DEV'\/>/<source dev='${DEV/$NAME/vhost}' mode='bridge'\/>/" "$XML"
+          if grep -qm1 "<interface type='bridge'>" "$XML"; then
+            # change to macvtap
+            log "change $DEV to macvtap in $XML"
+            sed -ri "${ROW[$i]} s/<interface type='bridge'>/<interface type='direct' trustGuestRxFilters='yes'>/; $ROW2 s/<source bridge='$DEV'\/>/<source dev='${DEV/$NAME/vhost}' mode='bridge'\/>/" "$XML"
+          fi
         else
-          # change to bridge
-          log "change $DEV to bridge in $XML"
-          sed -ri "${ROW[$i]} s/<interface type='direct'( trustGuestRxFilters='yes')?>/<interface type='bridge'>/; $ROW2 s/<source dev='$DEV' mode='bridge'\/>/<source bridge='${DEV/$NAME/br}'\/>/" "$XML"
+          if grep -qm1 "<interface type='direct'" "$XML"; then
+            # change to bridge
+            log "change $DEV to bridge in $XML"
+            sed -ri "${ROW[$i]} s/<interface type='direct'( trustGuestRxFilters='yes')?>/<interface type='bridge'>/; $ROW2 s/<source dev='$DEV' mode='bridge'\/>/<source bridge='${DEV/$NAME/br}'\/>/" "$XML"
+          fi
         fi
       fi
     done