Check password reuse in auth_passwd_check_passwd_available()

Change-Id: I5709d224930acfb7a33d99add6d136c67fcef187
Signed-off-by: jooseong lee <jooseong.lee@samsung.com>

diff --git a/src/include/auth-passwd.h b/src/include/auth-passwd.h
index 24be828c5937ac843da7799889e07fd69c98d581..4d3549daf78be5c861b9002fd8e4aa0f973ee53a 100644 (file)
--- a/src/include/auth-passwd.h
+++ b/src/include/auth-passwd.h
@@ -233,6 +233,8 @@ int auth_passwd_check_passwd_state(password_type passwd_type,
  * \return AUTH_PASSWD_API_SUCCESS
  * \return AUTH_PASSWD_API_ERROR_ACCESS_DENIED
  * \return AUTH_PASSWD_API_ERROR_SOCKET
+ * \return AUTH_PASSWD_API_ERROR_PASSWORD_REUSED
+ *              Typed string is an used password before
  * \return AUTH_PASSWD_API_ERROR_INVALID_MIN_LENGTH
  *              Typed string doesn't fulfill min length policy.
  * \return AUTH_PASSWD_API_ERROR_INVALID_MIN_COMPLEX_CHAR_NUM

diff --git a/src/server/service/password.cpp b/src/server/service/password.cpp
index 2175ee4487e064b5be640439ed9d30a4c1da5619..f165c7f072768edbe34ae60a763eda3c0393c269 100644 (file)
--- a/src/server/service/password.cpp
+++ b/src/server/service/password.cpp
@@ -141,9 +141,18 @@ int PasswordService::processCheckFunctions(PasswordHdrs hdr, MessageBuffer &buff
 
        case PasswordHdrs::HDR_CHK_PASSWD_AVAILABLE: {
                std::string challenge;
+               bool isPwdReused;
                Deserialization::Deserialize(buffer, passwdType);
                Deserialization::Deserialize(buffer, challenge);
-               result = m_policyManager.checkPolicy(passwdType, NO_PASSWORD, challenge, cur_user);
+               result = m_pwdManager.isPwdReused(passwdType, challenge, cur_user, isPwdReused);
+
+               if (result == AUTH_PASSWD_API_SUCCESS) {
+                       if(!isPwdReused)
+                               result = m_policyManager.checkPolicy(passwdType, NO_PASSWORD, challenge, cur_user);
+                       else
+                               result = AUTH_PASSWD_API_ERROR_PASSWORD_REUSED;
+               }
+
                break;
        }