The r variable was created unassigned and lately assigned to -1.
Even if r is always defined before its usage, sounds like initializing it to
what seems a default value (-1) from the beginning is safer for later code
inclusion.
Signed-off-by: Erwan Velu <erwan@redhat.com>
pg_t rawpg;
int64_t pool;
OSDMapRef curmap = service->get_osdmap();
- int r;
+ int r = -1;
string poolstr;
ss << "Invalid pool" << poolstr;
return;
}
- r = -1;
+
string objname, nspace;
cmd_getval(service->cct, cmdmap, "objname", objname);
std::size_t found = objname.find_first_of('/');