Skip to content
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

test/objectstore/test_bluefs: cleanups #17909

Merged
merged 2 commits into from Oct 24, 2017

Conversation

Projects
None yet
2 participants
@tchaikov
Copy link
Contributor

commented Sep 22, 2017

and avoid the false positive of coverity warning:

Use after free (USE_AFTER_FREE)

Signed-off-by: Kefu Chai kchai@redhat.com

tchaikov added some commits Sep 22, 2017

test/objectstore/test_bluefs: refactor related to open_for_write()
and avoid the false positive of coverity warning:

 Use after free (USE_AFTER_FREE)

Signed-off-by: Kefu Chai <kchai@redhat.com>
common/buffer: pass `const int[2]` to close_pipe()
be explicit instead of passing a `const int*`, this avoids the warning
from clang analyzer:

  Array subscript is undefined

Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov force-pushed the tchaikov:wip-coverity branch from e51b4d1 to 974f7ff Sep 22, 2017

@tchaikov tchaikov changed the title test/objectstore/test_bluefs: refactor write_single_file() test/objectstore/test_bluefs: cleanups Sep 22, 2017

@jdurgin jdurgin merged commit 184ca91 into ceph:master Oct 24, 2017

5 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details

@tchaikov tchaikov deleted the tchaikov:wip-coverity branch Oct 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.