The comment at line 92ff states "Check to see if >any< bridges exist[...]", while it actually checks for the hard-coded bridges "bridge0" and "bridge1". This causes iocell to always create those two bridges if present bridges on the system have other names.
Because the default route interface is being added to 'bridge0', the interface resets and drops all connections - on a jailhost this might be the default mgmt interface which carries the connection over which iocell was invoked and thus being terminated in the middle of starting the jail.
This Fix changes the grep'ing for bridge0|bridge1 to simply checking for interfaces belonging to the 'bridge' group via 'ifconfig -g bridge'.
Make sure to follow and check these boxes before submitting a PR! Thank you.
[x] Supply documentation according to CONTRIBUTING.md
bugfix; no update to documentation required
[x] Explain the feature
fixes the behavior of how iocell checks for bridges and decides if it creates the default bridge0 and bridge1
The comment at line 92ff states "Check to see if >any< bridges exist[...]", while it actually checks for the hard-coded bridges "bridge0" and "bridge1". This causes iocell to always create those two bridges if present bridges on the system have other names. Because the default route interface is being added to 'bridge0', the interface resets and drops all connections - on a jailhost this might be the default mgmt interface which carries the connection over which iocell was invoked and thus being terminated in the middle of starting the jail.
This Fix changes the grep'ing for bridge0|bridge1 to simply checking for interfaces belonging to the 'bridge' group via 'ifconfig -g bridge'.
Make sure to follow and check these boxes before submitting a PR! Thank you.
[x] Supply documentation according to CONTRIBUTING.md bugfix; no update to documentation required
[x] Explain the feature fixes the behavior of how iocell checks for bridges and decides if it creates the default bridge0 and bridge1
[x] Read CONTRIBUTING.md
[x] Only open the PR against the
developbranch.