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

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

    <bug>
          <bug_id>39420</bug_id>
          
          <creation_ts>2020-12-10 12:07:48 +0300</creation_ts>
          <short_desc>Polkit не проверяет список групп пользователя, назначенныx через NSS.</short_desc>
          <delta_ts>2021-01-26 23:59: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>polkit</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="Иван Савин">svn17</reporter>
          <assigned_to name="Valery Inozemtsev">shrek</assigned_to>
          <cc>aris</cc>
    
    <cc>ldv</cc>
    
    <cc>shaba</cc>
    
    <cc>shrek</cc>
    
    <cc>sin</cc>
    
    <cc>svn17</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>194797</commentid>
    <comment_count>0</comment_count>
    <who name="Иван Савин">svn17</who>
    <bug_when>2020-12-10 12:07:48 +0300</bug_when>
    <thetext>На текущий момент у нас для удалённых пользователей из домена не отрабатывает доступ к DBUS&apos;ным сервисам, даже если им назначается группа wheel.

Проблема в том, что служба polkit проверяет список групп, в которые входит пользователь без учёта того, какие группы пользователю назначаются через NSS. Поэтому локальные группы, назначаемые глобальным пользователям через libnss-role, не учитываются.

Необходимо доработать polkit, чтобы такая проверка для групп, назначаемых пользователю выполнялась.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>194798</commentid>
    <comment_count>1</comment_count>
    <who name="Иван Савин">svn17</who>
    <bug_when>2020-12-10 12:10:52 +0300</bug_when>
    <thetext>Вероятное решение:
http://git.altlinux.org/people/svn17/packages/?p=polkit.git;a=summary

merge-request в апстрим:
https://gitlab.freedesktop.org/polkit/polkit/-/merge_requests/71</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>194800</commentid>
    <comment_count>2</comment_count>
    <who name="Иван Савин">svn17</who>
    <bug_when>2020-12-10 12:53:06 +0300</bug_when>
    <thetext>Таска 263041</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>195123</commentid>
    <comment_count>3</comment_count>
    <who name="Dmitry V. Levin">ldv</who>
    <bug_when>2020-12-24 02:19:33 +0300</bug_when>
    <thetext>(In reply to Иван Савин from comment #1)
&gt; Вероятное решение:
&gt; http://git.altlinux.org/people/svn17/packages/?p=polkit.git;a=summary
&gt; 
&gt; merge-request в апстрим:
&gt; https://gitlab.freedesktop.org/polkit/polkit/-/merge_requests/71

Я не претендую на роль ревьюера polkit, но в функции addition_to_user_identities_user_of_subject у вас такая глубина вложенности блоков, что в тех проектах, где я исполняю роль ревьюера, я бы предложил использовать конструкции, которые не приводят к такой глубине вложенности.
Например,
if (unix_groups == NULL)
  return;
...
if (passwd == NULL)
  return;
и т.д.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>195145</commentid>
    <comment_count>4</comment_count>
    <who name="Иван Савин">svn17</who>
    <bug_when>2020-12-24 18:50:35 +0300</bug_when>
    <thetext>(Ответ для Dmitry V. Levin на комментарий #3) 
&gt; Я не претендую на роль ревьюера polkit, но в функции
&gt; addition_to_user_identities_user_of_subject у вас такая глубина вложенности
&gt; блоков, что в тех проектах, где я исполняю роль ревьюера, я бы предложил
&gt; использовать конструкции, которые не приводят к такой глубине вложенности.
&gt; Например,
&gt; if (unix_groups == NULL)
&gt;   return;
&gt; ...
&gt; if (passwd == NULL)
&gt;   return;
&gt; и т.д.

Исправил.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>195196</commentid>
    <comment_count>5</comment_count>
    <who name="Иван Савин">svn17</who>
    <bug_when>2020-12-30 14:38:41 +0300</bug_when>
    <thetext>(Ответ для Иван Савин на комментарий #4)
&gt; Исправил.

Это было сделано для другой ветки. Проделал эти же исправления для sisyphus.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>195763</commentid>
    <comment_count>6</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2021-01-26 23:59:04 +0300</bug_when>
    <thetext>polkit-0.118-alt2 -&gt; sisyphus:

 Mon Dec 07 2020 Ivan Savin &lt;svn17@altlinux&gt; 0.118-alt2
 - Add the ability to add user_of_subject to user_identities list
   if user_of_subject is a member of the group with administrator
   rights but it is not in /etc/groups (If a privileged group is
   assigned through the NSS). (closes: 39420)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>