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

common: buffer: critical bufferlist::zero bug #5388

Merged
merged 2 commits into from Sep 9, 2015

Conversation

Projects
None yet
4 participants
@smithfarm
Contributor

smithfarm commented Jul 28, 2015

yuyuyu101 added some commits Jul 9, 2015

buffer: Fix bufferlist::zero bug with special case
Fixes: #12252
Signed-off-by: Haomai Wang <haomaiwang@gmail.com>
(cherry picked from commit 43f583d)
UnittestBuffer: Add bufferlist zero test case
Signed-off-by: Haomai Wang <haomaiwang@gmail.com>
(cherry picked from commit 577acf6)

@smithfarm smithfarm self-assigned this Jul 28, 2015

@smithfarm smithfarm added this to the firefly milestone Jul 28, 2015

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Sep 4, 2015

@gregsfortytwo This has passed a rados suite (see http://tracker.ceph.com/issues/11644#rados for details). OK to merge, do you think?

@gregsfortytwo

This comment has been minimized.

Member

gregsfortytwo commented Sep 9, 2015

Yep, this looks good @smithfarm!

smithfarm added a commit that referenced this pull request Sep 9, 2015

Merge pull request #5388 from SUSE/wip-12490-firefly
buffer: critical bufferlist::zero bug

Reviewed-by: Gregory Farnum <gfarnum@redhat.com>

@smithfarm smithfarm merged commit 0a740d7 into ceph:firefly Sep 9, 2015

@smithfarm smithfarm deleted the SUSE:wip-12490-firefly branch Sep 9, 2015

@ghost ghost changed the title from buffer: critical bufferlist::zero bug to common: buffer: critical bufferlist::zero bug Oct 24, 2015

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