commit 31aeed84e85d503bb5f58155af101978daa7d96b
author Jeff Thompson <jefft0@gmail.com> Mon Nov 25 15:44:45 2013 -0800
committer Jeff Thompson <jefft0@gmail.com> Mon Nov 25 15:44:45 2013 -0800
tree 4048136dd7366787c742a056f5d36e657c137318
parent 728e2d6a02f031f8dd50fdb258a0a6af8424a377

security: PolicyManager: In checkVerificationPolicy, pass stepCount by value since the reference is not needed to modify the caller's value, and it is copied through to the ValidationRequest constructor.

src/security/policy/no-verify-policy-manager.cpp

diff --git a/src/security/policy/no-verify-policy-manager.cpp b/src/security/policy/no-verify-policy-manager.cpp
index e49ae75..38cc7fd 100644
--- a/src/security/policy/no-verify-policy-manager.cpp
+++ b/src/security/policy/no-verify-policy-manager.cpp
@@ -31,7 +31,7 @@
     
 shared_ptr<ValidationRequest>
 NoVerifyPolicyManager::checkVerificationPolicy
-  (const shared_ptr<Data>& data, const int& stepCount, const OnVerified& onVerified, const OnVerifyFailed& onVerifyFailed)
+  (const shared_ptr<Data>& data, int stepCount, const OnVerified& onVerified, const OnVerifyFailed& onVerifyFailed)
 { 
   onVerified(data); 
   return shared_ptr<ValidationRequest>();

src/security/policy/validation-request.hpp

diff --git a/src/security/policy/validation-request.hpp b/src/security/policy/validation-request.hpp
index d9facc9..386b45d 100644
--- a/src/security/policy/validation-request.hpp
+++ b/src/security/policy/validation-request.hpp
@@ -17,7 +17,7 @@
 public:
   ValidationRequest
     (const ptr_lib::shared_ptr<Interest> &interest, const OnVerified& onVerified, const OnVerifyFailed& onVerifyFailed,
-     const int& retry, const int& stepCount)
+     int retry, int stepCount)
   : interest_(interest), onVerified_(onVerified), onVerifyFailed_(onVerifyFailed), retry_(retry), stepCount_(stepCount)
   {
   }