diff options
| author | jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> | 2019-08-29 15:14:06 +0000 |
|---|---|---|
| committer | Vernon Mauery <vernon.mauery@linux.intel.com> | 2019-11-10 15:29:46 +0000 |
| commit | 9fc5fa180c3dc0a4fa5ed02efe4b990c40ba641d (patch) | |
| tree | 7d3dbda7d100345cb416c13076b599d91ba28e0b /user_channel/user_mgmt.hpp | |
| parent | 5aa2693770ea4bb17eb581ab9d5274e7a1edf862 (diff) | |
| download | phosphor-host-ipmid-9fc5fa180c3dc0a4fa5ed02efe4b990c40ba641d.tar.gz phosphor-host-ipmid-9fc5fa180c3dc0a4fa5ed02efe4b990c40ba641d.zip | |
user_mgmt:password authentication/update by PAM
Allowed password characters will be handled by password management,
instead of set user password command. This makes the checking to be
moved from set user password command to pam layer and accordingly
throw error, when the same is not valid.
Fix: So modified the code to handle with pam module itself.
Tested:
ipmitool user set password 6 0penBmc1\'
Set User Password command successful (user 6)
ipmitool user set password 6 0penBmc1\"
Set User Password command successful (user 6)
ipmitool user set password 6 12345678 //invalid password
IPMI command failed: Invalid data field in request
Set User Password command failed (user 6)
ipmitool user set password 3 asdf1234 //user id does not exit
IPMI command failed: Unspecified error
Set User Password command failed (user 3)
Signed-off-by: jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
Change-Id: Iba6e2c29a927d53e6ebdb5d32e83ecc7cbbd2fd0
Diffstat (limited to 'user_channel/user_mgmt.hpp')
| -rw-r--r-- | user_channel/user_mgmt.hpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/user_channel/user_mgmt.hpp b/user_channel/user_mgmt.hpp index 773b18d..0c38374 100644 --- a/user_channel/user_mgmt.hpp +++ b/user_channel/user_mgmt.hpp @@ -16,12 +16,11 @@ #pragma once #include "user_layer.hpp" -#include <ipmid/api.h> - #include <boost/interprocess/sync/file_lock.hpp> #include <boost/interprocess/sync/named_recursive_mutex.hpp> #include <cstdint> #include <ctime> +#include <ipmid/api.hpp> #include <sdbusplus/bus.hpp> #include <variant> |
