From 0174dc353febfead72810fba121a0b95d7edc661 Mon Sep 17 00:00:00 2001 From: Samveen Gulati Date: Tue, 1 Aug 2017 12:15:00 +0000 Subject: [PATCH] Fix bug introduced into `bmcsetup` by commit `bb460dc` Commit bb460dc introduced a minor bug, where the value of LOCKEDUSERS was no longer considered while handling existing entries in the user list. This fixes that by adding back the check. --- xCAT-genesis-scripts/bin/bmcsetup | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xCAT-genesis-scripts/bin/bmcsetup b/xCAT-genesis-scripts/bin/bmcsetup index d2c7668c8..fd88f331f 100755 --- a/xCAT-genesis-scripts/bin/bmcsetup +++ b/xCAT-genesis-scripts/bin/bmcsetup @@ -429,7 +429,7 @@ for user in $BMCUS; do if [ -z "$USERSLOT" ]; then USERSLOT=$((`ipmitool raw 6 0x44 1 1|awk '{print $3}'` + 1)) fi - if [ "$USERSLOT" == 0 ]; then USERSLOT=2; fi + if [ "$USERSLOT" == 0 -o -z "$LOCKEDUSERS" ]; then USERSLOT=2; fi if [ "$ISITE" = 1 ]; then allowcred.awk & CREDPID=$!