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

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

    <bug>
          <bug_id>29847</bug_id>
          
          <creation_ts>2014-02-25 14:07:46 +0400</creation_ts>
          <short_desc>Ошибка в драйвере bcmxcp</short_desc>
          <delta_ts>2014-04-23 19:21:17 +0400</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>nut-server</component>
          <version>unstable</version>
          <rep_platform>all</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>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alex Moskalenko">mav</reporter>
          <assigned_to name="Michael Shigorin">mike</assigned_to>
          <cc>ded</cc>
    
    <cc>mike</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>145484</commentid>
    <comment_count>0</comment_count>
    <who name="Alex Moskalenko">mav</who>
    <bug_when>2014-02-25 14:07:46 +0400</bug_when>
    <thetext>При работе драйвера bcmxcp с ИБП Eaton Powerware 9155 получаем segmentation fault. Отладка драйвера показала, что при получении общей информации об ИБП в функции upsdrv_initinfo() разработчики получают номера версий firmware для различных компонентов ИБП, и при этом сопоставляют его с заранее определенным массивом строк const char *cpu_name[5]. Данный ИБП сообщает о наличии 8 записей о версиях firmware. В цикле их расшифровки неправильно указано условие выхода из цикла (while ((len &gt; 0) &amp;&amp; (ncpu &lt;= 5))), что приводит к обращению к несуществующему элементу cpu_name[5] и, соответственно, sigsegv. Приложенный патч исправляет эту проблему, изменяя условие на ((len &gt; 0) &amp;&amp; (ncpu &lt;= 4)).

PS Возможно, есть смысл заканчивать цикл только по len &gt; 0, а вместо неопределенных (и, возможно, специфичных для каждой модели ИБП) наименований firmware подставлять что-то типа &quot;UNKN:&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>145485</commentid>
    <comment_count>1</comment_count>
      <attachid>6046</attachid>
    <who name="Alex Moskalenko">mav</who>
    <bug_when>2014-02-25 14:08:37 +0400</bug_when>
    <thetext>Created attachment 6046
Патч с исправлением</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>146318</commentid>
    <comment_count>2</comment_count>
    <who name="Michael Shigorin">mike</who>
    <bug_when>2014-04-23 19:10:50 +0400</bug_when>
    <thetext>Н-да, опять баг с патчем висит открытым, безобразие.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>146319</commentid>
    <comment_count>3</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2014-04-23 19:21:17 +0400</bug_when>
    <thetext>nut-2.6.5-alt5 -&gt; sisyphus:

* Wed Apr 23 2014 Michael Shigorin &lt;mike@altlinux&gt; 2.6.5-alt5
- added bcmxcp patch by Alex Moskalenko (closes: #29847)</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>6046</attachid>
            <date>2014-02-25 14:08:37 +0400</date>
            <delta_ts>2014-02-25 14:08:37 +0400</delta_ts>
            <desc>Патч с исправлением</desc>
            <filename>nut-2.6.5-bcmxcp.patch</filename>
            <type>text/plain</type>
            <size>294</size>
            <attacher name="Alex Moskalenko">mav</attacher>
            
              <data encoding="base64">LS0tIGEvZHJpdmVycy9iY214Y3AuYwkyMDEyLTExLTI2IDEyOjQ1OjI0LjAwMDAwMDAwMCArMDAw
MAorKysgYi9kcml2ZXJzL2JjbXhjcC5jCTIwMTQtMDItMjUgMDk6MzM6MjMuMDAwMDAwMDAwICsw
MDAwCkBAIC0xMTM0LDcgKzExMzQsNyBAQAogCQkJbGVuLS07CiAJCQluY3B1Kys7CiAKLQkJfSB3
aGlsZSAoKGxlbiA+IDApICYmIChuY3B1IDw9IDUpKTsKKwkJfSB3aGlsZSAoKGxlbiA+IDApICYm
IChuY3B1IDw9IDQpKTsKIAogCQlkc3RhdGVfc2V0aW5mbygidXBzLmZpcm13YXJlIiwgIiVzIiwg
cFRtcCk7CiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>