Commit a137875c authored by David Rich's avatar David Rich Committed by Rob Latham
Browse files

Take out "&" just to be sure copy constructor is called. May not be necessary...

Take out "&" just to be sure copy constructor is called. May not be necessary given the core dumps seem to be related to a BwTree bug.
parent 41251142
......@@ -332,7 +332,7 @@ static hg_return_t get_handler(hg_handle_t handle)
int value = 0;
if (values.size() == 1) {
std::vector<char> &data = values.front();
std::vector<char> data = values.front();
memcpy(&value, data.data(), data.size());
out.value = value;
out.ret = HG_SUCCESS;
......@@ -380,7 +380,7 @@ static hg_return_t bulk_get_handler(hg_handle_t handle)
// perhaps > 1 or 0 results in an error return value?
if (values.size() == 1) {
printf("SERVER: BULK GET: found 1 value for key=%lu\n", bgin.key);
std::vector<char> &data = values.front();
std::vector<char> data = values.front();
// will the transfer fit on the client side?
bgout.size = data.size();
if (bgout.size <= bgin.size) {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment