Project

General

Profile

Bug #39449

Uninit in EVP_DecryptFinal_ex on ceph::crypto::onwire::AES128GCM_OnWireRxHandler::authenticated_decrypt_update_final

Added by Sage Weil about 2 years ago. Updated almost 2 years ago.

Status:
Resolved
Priority:
Urgent
Category:
-
Target version:
-
% Done:

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(RADOS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

  <kind>UninitCondition</kind>
  <what>Conditional jump or move depends on uninitialised value(s)</what>
  <stack>
    <frame>
      <ip>0x1041AC7C</ip>
      <obj>/usr/lib64/libcrypto.so.1.0.2k</obj>
    </frame>
    <frame>
      <ip>0x10416BD6</ip>
      <obj>/usr/lib64/libcrypto.so.1.0.2k</obj>
      <fn>EVP_DecryptFinal_ex</fn>
    </frame>
    <frame>
      <ip>0x535F226</ip>
      <obj>/usr/lib64/ceph/libceph-common.so.0</obj>
      <fn>ceph::crypto::onwire::AES128GCM_OnWireRxHandler::authenticated_decrypt_update_final(ceph::buffer::v14_2_0::list&amp;&amp;, unsigned int)</fn>
      <dir>/usr/src/debug/ceph-15.0.0-606-g2739f47/src/msg/async</dir>
      <file>crypto_onwire.cc</file>
      <line>267</line>
    </frame>
    <frame>
      <ip>0x534E580</ip>
      <obj>/usr/lib64/ceph/libceph-common.so.0</obj>
      <fn>ProtocolV2::handle_read_frame_epilogue_main(std::unique_ptr&lt;ceph::buffer::v14_2_0::ptr_node, ceph::buffer::v14_2_0::ptr_node::disposer&gt;&amp;&amp;, int)</fn>
      <dir>/usr/src/debug/ceph-15.0.0-606-g2739f47/src/msg/async</dir>
      <file>ProtocolV2.cc</file>
      <line>1314</line>
    </frame>
    <frame>
      <ip>0x53368F3</ip>
      <obj>/usr/lib64/ceph/libceph-common.so.0</obj>
      <fn>ProtocolV2::run_continuation(Ct&lt;ProtocolV2&gt;&amp;)</fn>
      <dir>/usr/src/debug/ceph-15.0.0-606-g2739f47/src/msg/async</dir>
      <file>ProtocolV2.cc</file>
      <line>45</line>
    </frame>
    <frame>
      <ip>0x530464B</ip>
      <obj>/usr/lib64/ceph/libceph-common.so.0</obj>
      <fn>operator()</fn>
      <dir>/opt/rh/devtoolset-8/root/usr/include/c++/8/bits</dir>
      <file>std_function.h</file>
      <line>687</line>
    </frame>
    <frame>
      <ip>0x530464B</ip>
      <obj>/usr/lib64/ceph/libceph-common.so.0</obj>
      <fn>AsyncConnection::process()</fn>
      <dir>/usr/src/debug/ceph-15.0.0-606-g2739f47/src/msg/async</dir>
      <file>AsyncConnection.cc</file>
      <line>456</line>
    </frame>

/a/sage-2019-04-24_02:04:43-rados-wip-sage-testing-2019-04-23-1419-distro-basic-smithi/3883824
/a/sage-2019-04-24_02:04:43-rados-wip-sage-testing-2019-04-23-1419-distro-basic-smithi/3883684

History

#2 Updated by Kefu Chai about 2 years ago

/a/kchai-2019-04-27_02:20:42-rados-wip-kefu-testing-2019-04-26-2318-distro-basic-smithi/3898463/remote/smithi017/log/valgrind/mon.a.log.gz

#3 Updated by Neha Ojha about 2 years ago

  • Assignee set to Radoslaw Zarzynski

We probably need to backport this?

#4 Updated by Radoslaw Zarzynski almost 2 years ago

  • Status changed from 12 to Resolved
This has been backported with:

I believe we can put this one into the Resolved state. If the issue still persist/the stack trace for whitelisting needs to be tuned, let's use a new one.

Also available in: Atom PDF