The raw i2c command seems to populate the wdata buffer incorrectly when
the recently added bus= option is used, due to the way it calculates the
array index.  The following change should fix this behavior:

Index: lib/ipmi_raw.c
===================================================================
RCS file: /cvsroot/ipmitool/ipmitool/lib/ipmi_raw.c,v
retrieving revision 1.19
diff -u -p -r1.19 ipmi_raw.c
--- lib/ipmi_raw.c      12 Jul 2006 19:31:31 -0000      1.19
+++ lib/ipmi_raw.c      2 Jul 2007 21:04:20 -0000
@@ -225,7 +225,7 @@ ipmi_rawi2c_main(struct ipmi_intf * intf
        memset(wdata, 0, IPMI_I2C_MASTER_MAX_SIZE);
        for (; i < argc; i++) {
                uint8_t val = (uint8_t)strtol(argv[i], NULL, 0);
-               wdata[i-2] = val;
+               wdata[wsize] = val;
                wsize++;
        }

Thanks,
Joshua Neal

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Ipmitool-devel mailing list
Ipmitool-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ipmitool-devel

Reply via email to