'ipmitool user priv X Y Z' correctly sets privilege level of an user, but it
also enables IPMI messaging for the user. IMHO the command should do only
one thing, there is 'ipmitool channel setaccess X Y ipmi=[on|off]' for
enabling/disabling the IPMI messaging.

Signed-off-by: Jan Safranek <jsafr...@redhat.com>
---

 lib/ipmi_user.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/lib/ipmi_user.c b/lib/ipmi_user.c
index 57e64b7..25fd643 100644
--- a/lib/ipmi_user.c
+++ b/lib/ipmi_user.c
@@ -359,7 +359,6 @@ ipmi_user_set_userpriv(
 
        /* The channel number will remain constant throughout this function */
        msg_data[0] = (channel   & 0x0f);
-       msg_data[0] |= 0x90;            /* enable ipmi messaging */
        msg_data[1] = (user_id   & 0x3f);
        msg_data[2] = (privLevel & 0x0f);
        msg_data[3] = 0;


------------------------------------------------------------------------------
This SF.net email is sponsored by Sprint
What will you do first with EVO, the first 4G phone?
Visit sprint.com/first -- http://p.sf.net/sfu/sprint-com-first
_______________________________________________
Ipmitool-devel mailing list
Ipmitool-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ipmitool-devel

Reply via email to