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

rgw_rados: initialize cur_shard #15735

Merged
merged 1 commit into from Jun 16, 2017

Conversation

Projects
None yet
3 participants
@theanalyst
Member

theanalyst commented Jun 16, 2017

valgrind reported an uninitialized value on use of cur_shard at a few
places, so explicitly declaring this to 0 as we are incrementing this
later

Signed-off-by: Abhishek Lekshmanan abhishek@suse.com

rgw_rados: initialize cur_shard
valgrind reported an uninitialized value on use of cur_shard at a few
places, so explicitly declaring this to 0 as we are incrementing this
later

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

@theanalyst theanalyst referenced this pull request Jun 16, 2017

Merged

rgw: implement get/put object tags for S3 #13753

13 of 13 tasks complete
@@ -3486,7 +3486,7 @@ class RGWIndexCompletionManager {
int num_shards;
std::atomic<int> cur_shard;
std::atomic<int> cur_shard {0};

This comment has been minimized.

@chardan

chardan Jun 16, 2017

Contributor

Good catch!

@cbodley cbodley merged commit 84de9cc into ceph:master Jun 16, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
arm64 make check arm64 make check succeeded
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment