Bug 738484 - clvmd hangs/crash if invalid local request sent
clvmd hangs/crash if invalid local request sent
Status: CLOSED ERRATA
Product: Red Hat Enterprise Linux 6
Classification: Red Hat
Component: lvm2 (Show other bugs)
6.1
Unspecified Unspecified
unspecified Severity unspecified
: rc
: ---
Assigned To: Milan Broz
Corey Marthaler
:
Depends On:
Blocks: 756082
  Show dependency treegraph
 
Reported: 2011-09-14 19:20 EDT by Roman
Modified: 2013-02-28 23:10 EST (History)
10 users (show)

See Also:
Fixed In Version: lvm2-2.02.95-1.el6
Doc Type: Bug Fix
Doc Text:
Previously, if clvmd received invalid request through its socket (for example an incomplete header sent), clvmd process could crash or stay in infinite loop. Additional checks were added so such invalid packet now causes proper error reply to client and clvmd no longer crashes.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2012-06-20 10:59:53 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
clvmd patch (1.60 KB, patch)
2011-09-14 19:21 EDT, Roman
no flags Details | Diff

  None (edit)
Description Roman 2011-09-14 19:20:47 EDT
Description of problem:
clvmd hangs or crash if invalid request sent via /var/run/lvm/clvmd.sock

Version-Release number of selected component (if applicable):
lvm2-cluster-2.02.83-3.el6.x86_64

How reproducible:
always

Steps to Reproduce:
1. start clvmd
2. run perl -e 'print "\xff"x32' | socat STDIN UNIX-CONNECT:/var/run/lvm/clvmd.sock
  
Actual results:
clvmd crash or hangs and eat 100% of CPU

Expected results:
no hang or crash occurs

Additional info:
invalid request may send lvm in case of memory pressure or similar problem.
attached patch solved problem for me.
Comment 1 Roman 2011-09-14 19:21:57 EDT
Created attachment 523277 [details]
clvmd patch
Comment 3 Milan Broz 2011-09-15 07:32:41 EDT
Yes, this seem to be reasonable fix, thanks.
Comment 4 Milan Broz 2011-09-16 10:41:29 EDT
Slightly modified patch committed upstream, thanks.
Comment 10 Milan Broz 2012-04-24 14:18:45 EDT
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Previously, if clvmd received invalid request through its socket (for example an incomplete header sent), clvmd process could crash or stay in infinite loop.
Additional checks were added so such invalid packet now causes proper error reply to client and clvmd no longer crashes.
Comment 11 Corey Marthaler 2012-04-26 17:52:47 EDT
Marking verified (SanityOnly).
Comment 13 errata-xmlrpc 2012-06-20 10:59:53 EDT
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2012-0962.html

Note You need to log in before you can comment on or make changes to this bug.