ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | Typos in the write_systemd_networkd_bootproto() function | ||
|---|---|---|---|
| Product: | Sisyphus | Reporter: | Nikolay A. Fetisov <naf> |
| Component: | alterator-net-functions | Assignee: | Mikhail Efremov <sem> |
| Status: | CLOSED FIXED | QA Contact: | qa-sisyphus |
| Severity: | minor | ||
| Priority: | P5 | CC: | boyarsh, sem, zerg |
| Version: | unstable | ||
| Hardware: | all | ||
| OS: | Linux | ||
В alterator-net-functions 2.1.1-alt1 в функции write_systemd_networkd_bootproto() есть блоки кода добавляющие в network-файл параметры Address (строки 1173-1181) и Gatewaу (строки 1183-1191). Блок кода для Address - -----------8<-------------- first_value_flag= for addr in $ipv4_addresses $ipv6_addresses; do if [ -n "$first_value_flag" ]; then write_systemd_networkd_ini_option "$network_filename" "Network" "Address" "$addr" first_value_flag=1 else add_systemd_networkd_ini_option "$network_filename" "Network" "Address" "$addr" fi done -----------8<-------------- По логика кода предполагается, что для первого адреса из списка будет вызвана функция write_systemd_networkd_ini_option(), для последующих - add_systemd_networkd_ini_option(). Но, т.к. начальное значиние first_value_flag - пустое, то всегда вызывается add_systemd_networkd_ini_option(). Аналогичное поведение ниже в строках 1183-1191 для Gatewaу. По-видимому, имелось в виду -----------8<-------------- first_value_flag=1 for ...; do if [ -n "$first_value_flag" ]; then ... first_value_flag= else ... fi done -----------8<-------------- На работе кода это, впрочем, не сказывается - для первого добавляемого параметра поведение add_systemd_networkd_ini_option() такое же, как у write_systemd_networkd_ini_option().