[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] a865ce: blockjob: leak fix, remove from txn w
From: |
GitHub |
Subject: |
[Qemu-commits] [qemu/qemu] a865ce: blockjob: leak fix, remove from txn when failing e... |
Date: |
Wed, 04 Apr 2018 05:59:38 -0700 |
Branch: refs/heads/master
Home: https://github.com/qemu/qemu
Commit: a865cebb8285cab5896dca9d3c5ad805cbbd2b9d
https://github.com/qemu/qemu/commit/a865cebb8285cab5896dca9d3c5ad805cbbd2b9d
Author: Marc-André Lureau <address@hidden>
Date: 2018-04-03 (Tue, 03 Apr 2018)
Changed paths:
M blockjob.c
Log Message:
-----------
blockjob: leak fix, remove from txn when failing early
This fixes leaks found by ASAN such as:
GTESTER tests/test-blockjob
=================================================================
==31442==ERROR: LeakSanitizer: detected memory leaks
Direct leak of 24 byte(s) in 1 object(s) allocated from:
#0 0x7f88483cba38 in __interceptor_calloc (/lib64/libasan.so.4+0xdea38)
#1 0x7f8845e1bd77 in g_malloc0 ../glib/gmem.c:129
#2 0x7f8845e1c04b in g_malloc0_n ../glib/gmem.c:360
#3 0x5584d2732498 in block_job_txn_new /home/elmarco/src/qemu/blockjob.c:172
#4 0x5584d2739b28 in block_job_create /home/elmarco/src/qemu/blockjob.c:973
#5 0x5584d270ae31 in mk_job /home/elmarco/src/qemu/tests/test-blockjob.c:34
#6 0x5584d270b1c1 in do_test_id
/home/elmarco/src/qemu/tests/test-blockjob.c:57
#7 0x5584d270b65c in test_job_ids
/home/elmarco/src/qemu/tests/test-blockjob.c:118
#8 0x7f8845e40b69 in test_case_run ../glib/gtestutils.c:2255
#9 0x7f8845e40f29 in g_test_run_suite_internal ../glib/gtestutils.c:2339
#10 0x7f8845e40fd2 in g_test_run_suite_internal ../glib/gtestutils.c:2351
#11 0x7f8845e411e9 in g_test_run_suite ../glib/gtestutils.c:2426
#12 0x7f8845e3fe72 in g_test_run ../glib/gtestutils.c:1692
#13 0x5584d270d6e2 in main /home/elmarco/src/qemu/tests/test-blockjob.c:377
#14 0x7f8843641f29 in __libc_start_main (/lib64/libc.so.6+0x20f29)
Add an assert to make sure that the job doesn't have associated txn before
free().
[Jeff Cody: N.B., used updated patch provided by John Snow]
Signed-off-by: Marc-André Lureau <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>
Commit: 604343ced7288cd2087ba1a9ee855c100775ba65
https://github.com/qemu/qemu/commit/604343ced7288cd2087ba1a9ee855c100775ba65
Author: Marc-André Lureau <address@hidden>
Date: 2018-04-03 (Tue, 03 Apr 2018)
Changed paths:
M blockjob.c
Log Message:
-----------
blockjob: use qapi enum helpers
QAPI generator provide #define helpers for looking up enum string.
Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: John Snow <address@hidden>
Message-id: address@hidden
Signed-off-by: Jeff Cody <address@hidden>
Commit: 9dae635afa98f83688806861cefe77ff1b4d76a8
https://github.com/qemu/qemu/commit/9dae635afa98f83688806861cefe77ff1b4d76a8
Author: Kevin Wolf <address@hidden>
Date: 2018-04-03 (Tue, 03 Apr 2018)
Changed paths:
M block/gluster.c
Log Message:
-----------
gluster: Fix blockdev-add with server.N.type=unix
The legacy command line interface gets the socket path from an option
called 'socket'. QAPI in contract uses SocketAddress, where the
corresponding option is called 'path'.
Fix the gluster block driver to accept both 'socket' and 'path', with
'path' being the preferred syntax.
https://bugzilla.redhat.com/show_bug.cgi?id=1545155
Cc: address@hidden
Signed-off-by: Kevin Wolf <address@hidden>
Message-id: address@hidden
Signed-off-by: Jeff Cody <address@hidden>
Commit: e5efa1f5f2d32cbfbf18ba84300736503985b593
https://github.com/qemu/qemu/commit/e5efa1f5f2d32cbfbf18ba84300736503985b593
Author: Peter Maydell <address@hidden>
Date: 2018-04-04 (Wed, 04 Apr 2018)
Changed paths:
M block/gluster.c
M blockjob.c
Log Message:
-----------
Merge remote-tracking branch 'remotes/cody/tags/block-pull-request' into
staging
# gpg: Signature made Tue 03 Apr 2018 17:10:22 BST
# gpg: using RSA key BDBE7B27C0DE3057
# gpg: Good signature from "Jeffrey Cody <address@hidden>"
# gpg: aka "Jeffrey Cody <address@hidden>"
# gpg: aka "Jeffrey Cody <address@hidden>"
# Primary key fingerprint: 9957 4B4D 3474 90E7 9D98 D624 BDBE 7B27 C0DE 3057
* remotes/cody/tags/block-pull-request:
gluster: Fix blockdev-add with server.N.type=unix
blockjob: use qapi enum helpers
blockjob: leak fix, remove from txn when failing early
Signed-off-by: Peter Maydell <address@hidden>
Compare: https://github.com/qemu/qemu/compare/094b62cd9cd9...e5efa1f5f2d3
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-commits] [qemu/qemu] a865ce: blockjob: leak fix, remove from txn when failing e...,
GitHub <=