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: fix uninitialized fields #14120

Merged
merged 1 commit into from Apr 3, 2017

Conversation

Projects
None yet
5 participants
@wangzhengyong
Contributor

wangzhengyong commented Mar 24, 2017

@joscollin

The fix looks good. But no idea why the build failed. It successfully builds in my PC.

@wangzhengyong

This comment has been minimized.

Contributor

wangzhengyong commented Mar 26, 2017

retest it please.

bool done;
int ret;
bool done = false;
int ret = 0;

This comment has been minimized.

@yehudasa

yehudasa Mar 30, 2017

Member

@wangzhengyong no need to initialize ret, it's assigned at the next line

rgw: fix uninitialized fields
Signed-off-by: wangzhengyong@cmss.chinamobile.com
@oritwas

This comment has been minimized.

Contributor

oritwas commented Apr 3, 2017

jenkins test this please

@yehudasa

lgtm

@yehudasa yehudasa merged commit e18b2f2 into ceph:master Apr 3, 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment