diff options
| author | Yehuda Sadeh <yehuda@inktank.com> | 2013-09-18 10:37:21 -0700 |
|---|---|---|
| committer | Yehuda Sadeh <yehuda@inktank.com> | 2013-09-18 10:37:21 -0700 |
| commit | e2807fc2f9e8c8ca7977a8ef1295bb126f557e9e (patch) | |
| tree | 7965a37f1e6d668e737ac489aeee3f8ba9ef1050 | |
| parent | 4216eac0f59af60f60d4ce909b9ace87a7b64ccc (diff) | |
| download | ceph-e2807fc2f9e8c8ca7977a8ef1295bb126f557e9e.tar.gz | |
rgw: use bufferlist::append() instead of bufferlist::push_back()wip-6340
push_back() expects char *, whereas append can append a single char.
Appending a NULL char to push_back is cast as a NULL pointer which is
bad.
Signed-off-by: Yehuda Sadeh <yehuda@inktank.com>
| -rw-r--r-- | src/rgw/rgw_rados.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/rgw/rgw_rados.cc b/src/rgw/rgw_rados.cc index bada7d22d1b..5ec61b965a1 100644 --- a/src/rgw/rgw_rados.cc +++ b/src/rgw/rgw_rados.cc @@ -2616,7 +2616,7 @@ int RGWRados::copy_obj(void *ctx, { /* opening scope so that we can do goto, sorry */ bufferlist& extra_data_bl = processor.get_extra_data(); if (extra_data_bl.length()) { - extra_data_bl.push_back((char)0); + extra_data_bl.append((char)0); JSONParser jp; if (!jp.parse(extra_data_bl.c_str(), extra_data_bl.length())) { ldout(cct, 0) << "failed to parse response extra data. len=" << extra_data_bl.length() << " data=" << extra_data_bl.c_str() << dendl; |