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

os/bluestore: avoid nullptr in bluestore_extent_ref_map_t::bound_encode #14073

Merged
merged 1 commit into from Mar 24, 2017

Conversation

Projects
None yet
2 participants
@liewegas
Member

liewegas commented Mar 21, 2017

Signed-off-by: Sage Weil sage@redhat.com

@liewegas liewegas requested a review from tchaikov Mar 21, 2017

Show outdated Hide outdated src/os/bluestore/bluestore_types.h
p += elem_size * ref_map.size();
if (!ref_map.empty()) {
size_t elem_size = 0;
denc_varint_lowz((uint32_t)0, p);

This comment has been minimized.

@tchaikov

tchaikov Mar 22, 2017

Contributor

s/p/elem_size/,

and it should be uint64_t.

@tchaikov

tchaikov Mar 22, 2017

Contributor

s/p/elem_size/,

and it should be uint64_t.

os/bluestore: avoid nullptr in bluestore_extent_ref_map_t::bound_encode
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Mar 23, 2017

Member

@tchaikov builds now :)

Member

liewegas commented Mar 23, 2017

@tchaikov builds now :)

@liewegas liewegas merged commit 8275fda into ceph:master Mar 24, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@liewegas liewegas deleted the liewegas:wip-bluestore-nullptr branch Mar 24, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment