ALT Linux Bugzilla – #58359

Версия пакета:
	ip-brctl-0.2-alt1

Согласно man'у:
	The  command  brctl  addif  <brname> <ifname> will make the interface <ifname> a port of the bridge <brname>.

Шаги воспроизведения:
	# brctl addif br0 ens19

Результат:
	device br0 is a bridge device itself; can't enslave a bridge device to a bridge device.

Ошибка в функции cmd_addif():
	1. Проверка на то, что добавляемый интерфейс уже является членом другого моста, выполняется для первого аргумента (моста), а должна для второго (интерфейса).
	2. Проверка на то, что добавляемый интерфейс не является мостом, также выполняется для первого аргумента, хотя должна для второго.

Исправление:
--- a/usr/sbin/ip-brctl
+++ b/usr/sbin/ip-brctl
@@ -338,10 +338,10 @@ cmd_delbr() {
 cmd_addif() {
        err_dev_exists n "${1}" "bridge ${1} does not exist!"
        err_dev_exists n "${2}" "interface ${2} does not exist!"
-       if [ -n "$(parse_iplink master "${1}")" ]; then
+       if [ -n "$(parse_iplink master "${2}")" ]; then
                err "device ${2} is already a member of a bridge; can't enslave it to bridge ${1}."
        fi
-       err_dev_exists y "${1}" "device ${1} is a bridge device itself; can't enslave a bridge device to a bridge device." bridge
+       err_dev_exists y "${2}" "device ${2} is a bridge device itself; can't enslave a bridge device to a bridge device." bridge
 
        exec_iplink "set ${2} master ${1}" "can't add ${2} to bridge ${1}"
 }