auth/KeyRing: do not decode a copy of bl

i checked all the code paths calling into KeyRing::decode(), none of
them relies on the behavior that the bl is not mutated after the
iterator is decoded.

actually, it is more intuitive to always move the iterator forward when
decoding the encoded keyring in the bufferlist.

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/auth/KeyRing.cc b/src/auth/KeyRing.cc
index 6686fa9f51af418361172bdf491622a68bc9474b..d2e4cd391cf9e63893725fdbf41a2b5a2158bcc2 100644 (file)
--- a/src/auth/KeyRing.cc
+++ b/src/auth/KeyRing.cc
@@ -203,8 +203,7 @@ void KeyRing::decode_plaintext(bufferlist::const_iterator& bli)
 
 void KeyRing::decode(bufferlist::const_iterator& bl)
 {
-  auto start_pos = bl;
-  decode_plaintext(start_pos);
+  decode_plaintext(bl);
 }
 
 int KeyRing::load(CephContext *cct, const std::string &filename)