diff options
author | arun-pm <arun.p.m@linux.intel.com> | 2020-01-10 15:18:04 +0530 |
---|---|---|
committer | Tom Joseph <tomjoseph@in.ibm.com> | 2020-01-22 18:02:26 +0000 |
commit | bbe728c374d2583596e628c88c0d8110836f021d (patch) | |
tree | a964dd06e7163896765ef552e11fb0c219d58322 /user_channel/user_mgmt.hpp | |
parent | 369406e6c157a6656a08331cc5fb85313ba7154e (diff) | |
download | phosphor-host-ipmid-bbe728c374d2583596e628c88c0d8110836f021d.tar.gz phosphor-host-ipmid-bbe728c374d2583596e628c88c0d8110836f021d.zip |
[user-mgmt]: Rearrange signal handler registration
Currently, signal handler are registered after the GetManagedObjects
query. This may miss any signals generated in-between this time. Correct
flow must be to register for the signal first, followed by
GetManagedObjects query.
Tested:
1. Verified that user list are properly listed after this change.
Note: Also, performed following tests to determine the order didn't cause any
problem. Verified with both ipmi_user.json deleted and preserved.
2. Started phosphor-user-manager after host-ipmid.
3. Re-started phosphor-user-manager.
4. Started phosphor-host-ipmid after phosphor-user-manager.
5. Restarted phosphor-host-ipmid.
Change-Id: I124b5e96672e0456289bca7a2b889e4b897c0545
Signed-off-by: arun-pm <arun.p.m@linux.intel.com>
Diffstat (limited to 'user_channel/user_mgmt.hpp')
-rw-r--r-- | user_channel/user_mgmt.hpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/user_channel/user_mgmt.hpp b/user_channel/user_mgmt.hpp index 0c38374..054c0e2 100644 --- a/user_channel/user_mgmt.hpp +++ b/user_channel/user_mgmt.hpp @@ -392,8 +392,9 @@ class UserAccess void getSystemPrivAndGroups(); /** @brief function to init user data from configuration & D-Bus objects + * and to register for signals * */ - void initUserDataFile(); + void cacheUserDataFile(); }; } // namespace ipmi |