Bug 49159 - Typos in the write_systemd_networkd_bootproto() function
Summary: Typos in the write_systemd_networkd_bootproto() function
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-net-functions (show other bugs)
Version: unstable
Hardware: all Linux
: P5 minor
Assignee: Mikhail Efremov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-01-23 15:40 MSK by Nikolay A. Fetisov
Modified: 2024-01-23 21:19 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolay A. Fetisov 2024-01-23 15:40:58 MSK
В 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().
Comment 1 Repository Robot 2024-01-23 21:19:02 MSK
alterator-net-functions-2.1.2-alt1 -> sisyphus:

 Tue Jan 23 2024 Mikhail Efremov <sem@altlinux> 2.1.2-alt1
 - Fix write_systemd_networkd_bootproto() (closes: #49159).
 - Fix sed regex for systemd-networkd config (closes: #49158).