Bug 1309439
| Summary: | libtevent leaks memory during signal handling | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 7 | Reporter: | Jakub Hrozek <jhrozek> | ||||
| Component: | libtevent | Assignee: | Jakub Hrozek <jhrozek> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Steeve Goveas <sgoveas> | ||||
| Severity: | urgent | Docs Contact: | |||||
| Priority: | urgent | ||||||
| Version: | 7.3 | CC: | apeetham, asn, enewland, gagriogi, mkolaja, msauton, pbokoc, pbrezina, sgoveas, tscherf | ||||
| Target Milestone: | rc | Keywords: | ZStream | ||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | libtevent-0.9.26-2.el7 | Doc Type: | Bug Fix | ||||
| Doc Text: |
Previously, the libtevent package suffered from a memory leak in signal handling. This bug has been fixed and memory leaks no longer occur in libtevent.
|
Story Points: | --- | ||||
| Clone Of: | |||||||
| : | 1309440 1324387 (view as bug list) | Environment: | |||||
| Last Closed: | 2016-11-04 07:07:16 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 1309440, 1324387 | ||||||
| Attachments: |
|
||||||
|
Description
Jakub Hrozek
2016-02-17 20:00:02 UTC
Steps to reproduce are wrong. It is not about receiving the signals, the leak occurs on registering a signal handler. I'm attaching a simple reproducer in C. Created attachment 1128225 [details]
reproducer
Steeve, do you think we can manage to have this in Z-stream? (In reply to gagriogi from comment #5) > Steeve, do you think we can manage to have this in Z-stream? We tentatively agreed to include this in U4. Pavel, can you please create some reproducer (C code would be fine) that QE could use to reproduce and verify the bug? The reproducer was attached in February :-) https://bugzilla.redhat.com/show_bug.cgi?id=1309439#c3 To reproduce, grab the source file that Pavel attached as comment #3 and save it as some file, I used tev.c as an example. Make sure that libtevent-devel and libtalloc-devel are installed. Then compile the source with: $ gcc -ltevent -ltalloc tev.c and run the resulting binary: $./a.out With the unpatched packages, you should see a bunch of "struct sigaction" structures being reported by libtalloc as leaked. You should not see these leaks with the patched version. Steeve, can you add a qa_ack based on comment #12 as a reproducer? Verified the bug on libtevent Version: libtevent-0.9.28-1.el7.x86_64 Steps followed during verification: 1. Already reproduced this bug with the older build: libtevent-0.9.25-1.el7.x86_64 during verification of RHEL-7.2.Z bug BZ1324387. 2. With the patched build, the "struct sigaction" shouldn't be reported, see the output: # ./a.out full talloc report on 'null_context' (total 2928 bytes in 7 blocks) struct tevent_sig_state contains 2768 bytes in 1 blocks (ref 0) 0x1a53680 struct tevent_ops_list contains 32 bytes in 1 blocks (ref 0) 0x1a53320 struct tevent_ops_list contains 32 bytes in 1 blocks (ref 0) 0x1a53290 struct tevent_ops_list contains 32 bytes in 1 blocks (ref 0) 0x1a53200 struct tevent_ops_list contains 32 bytes in 1 blocks (ref 0) 0x1a53170 struct tevent_ops_list contains 32 bytes in 1 blocks (ref 0) 0x1a530e0 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. https://rhn.redhat.com/errata/RHEA-2016-2474.html |