The check in interfaces_bridge.php to prevent deleting an assigned bridge does not work.
In the bridge_inuse function, $a_bridges seems to be out of scope, so it is not accessible when the function attempts to compare against a value in it, thus the comparison always fails. This patch modifies it to accept the name of the interface directly, so it does not try to use an external variable. The call to the function has been adjusted accordingly.
#2 Updated by Erik Fonnesbeck over 9 years ago
- File 0001-Revert-Fix-the-check-for-whether-the-bridge-is-assi.patch 0001-Revert-Fix-the-check-for-whether-the-bridge-is-assi.patch added
- File 0002-Fix-scope-problem-in-_inuse-functions-on-some-of-th.patch 0002-Fix-scope-problem-in-_inuse-functions-on-some-of-th.patch added
This set of patches reverts the style of the function to the previous style and then fixes the scope problem on that instead. It also fixes the same issue on four of the other pages.