New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hammer: ceph-osd valgrind invalid reads/writes #9238
Conversation
test this please |
test this please |
1 similar comment
test this please |
Otherwise, if it is reaped while we are waiting, it'll be a use-after-free. Fixes: http://tracker.ceph.com/issues/15870 Signed-off-by: Samuel Just <sjust@redhat.com> (cherry picked from commit b224912) Conflicts: src/msg/simple/Pipe.cc: nullptr changed to 0 because hammer is not compiled with -std=c++11
b02986f
to
055427c
Compare
Reviewed-by: Nathan Cutler <ncutler@suse.com>
Reviewed-by: Nathan Cutler <ncutler@suse.com>
Reviewed-by: Nathan Cutler <ncutler@suse.com>
@athanatos This PR is in the latest round of hammer-backports integration tests, which passed a rados run (the only failures are a valgrind false positive that has since been fixed by ceph/teuthology#915 and http://tracker.ceph.com/issues/15139 which is an infrastructure issue with two of the tests) - for details, see: http://tracker.ceph.com/issues/15895#note-18 Do you think this PR is OK to merge? |
lgtm |
http://tracker.ceph.com/issues/15962