<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>49158</bug_id>
          
          <creation_ts>2024-01-23 15:17:15 +0300</creation_ts>
          <short_desc>Misbehavior in the sed regex inside add_systemd_networkd_ini_option() function</short_desc>
          <delta_ts>2024-01-23 21:20:04 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>alterator-net-functions</component>
          <version>unstable</version>
          <rep_platform>x86_64</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P5</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Nikolay A. Fetisov">naf</reporter>
          <assigned_to name="Mikhail Efremov">sem</assigned_to>
          <cc>boyarsh</cc>
    
    <cc>sem</cc>
    
    <cc>zerg</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>240481</commentid>
    <comment_count>0</comment_count>
    <who name="Nikolay A. Fetisov">naf</who>
    <bug_when>2024-01-23 15:17:15 +0300</bug_when>
    <thetext>Для alterator-net-function 2.1.1-alt1 в функции add_systemd_networkd_ini_option()
для добавления дополнительного значения параметра в network-файл после уже там
имеющегося используется sed вида (строка 1089)

sed -i -e &quot;1,// s/^\(\s*\)\($latest\)$/\1\2\n\1$3 = $4/&quot; -- &quot;$1&quot;

В функции write_systemd_networkd_bootproto() (строка 1179) эта функция вызывается
для добавления в network-файлов адресов IP - которые указываются с маской.
В sed как разделитель для оператора замены &apos;s&apos; используется &apos;/&apos; - после подставления
в строку запуска sed адреса IP получается синтаксическая ошибка.

Пример для проверки:
------------8&lt;--------------
#!/bin/sh

. alterator-net-functions

network_filename=$(mktemp /tmp/XXXX.network)
for addr in 10.0.0.1/8 10.0.0.2/8 10.0.0.3/8 fd00:1/64 fd00:2/64; do
        echo &quot;ADDR=$addr&quot;
        add_systemd_networkd_ini_option &quot;$network_filename&quot; &quot;Network&quot; &quot;Address&quot; &quot;$addr&quot;
done

echo
echo &quot;Network file:&quot;
cat $network_filename
rm -f &quot;$network_filename&quot;
------------8&lt;--------------

Результат запуска:
------------8&lt;--------------
ADDR=10.0.0.1/8
ADDR=10.0.0.2/8
sed: -e expression #1, char 42: unknown option to `s&apos;
ADDR=10.0.0.3/8
sed: -e expression #1, char 42: unknown option to `s&apos;
ADDR=fd00:1/64
sed: -e expression #1, char 42: unknown option to `s&apos;
ADDR=fd00:2/64
sed: -e expression #1, char 42: unknown option to `s&apos;

Network file:
[Network]
        Address = 10.0.0.1/8
------------8&lt;--------------

В network-файл добавился только первый адрес, т.к. для него в add_systemd_networkd_ini_option() строка с sed не использовалась,
остальные вызвали ошибки.

По-видимому, надо заменить разделитель на что-либо другое - как вариант, на &apos; .
Хотя это зависит от того, где используется эта функция - но в alterator-net-function
через неё записываются только Address, Gateway и DNS - в них кавычек быть не может
(% и много другое может быть в DNS как часть адреса IPv6).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>240511</commentid>
    <comment_count>1</comment_count>
    <who name="Mikhail Efremov">sem</who>
    <bug_when>2024-01-23 20:30:45 +0300</bug_when>
    <thetext>Спасибо за подробный анализ.
&gt; По-видимому, надо заменить разделитель на что-либо другое - как вариант, на
&gt; &apos; .
&gt; Хотя это зависит от того, где используется эта функция - но в
&gt; alterator-net-function
&gt; через неё записываются только Address, Gateway и DNS - в них кавычек быть не
&gt; может
&gt; (% и много другое может быть в DNS как часть адреса IPv6).

Думаю | вполне подойдет.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>240516</commentid>
    <comment_count>2</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2024-01-23 21:20:04 +0300</bug_when>
    <thetext>alterator-net-functions-2.1.2-alt1 -&gt; sisyphus:

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

    </bug>

</bugzilla>