From 66fcf484a4f6248932a13e6c5df4a06d6acd8c98 Mon Sep 17 00:00:00 2001
From: Zdenek Styblik <zdenek.styblik@gmail.com>
Date: Fri, 29 Mar 2013 14:52:59 +0000
Subject: [PATCH] ID: 3600911 - 'lib/ipmi_fru.c' - fix multiple increments in
 args to printf

Commit fixes multiple increments in arguments of printf() which leads to
undefined/inconsistent behaviour and results.

Reported-by: Ales Ledvinka
---
 ipmitool/lib/ipmi_fru.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/ipmitool/lib/ipmi_fru.c b/ipmitool/lib/ipmi_fru.c
index 3ac6afb..7177b3a 100644
--- a/ipmitool/lib/ipmi_fru.c
+++ b/ipmitool/lib/ipmi_fru.c
@@ -2236,7 +2236,8 @@ static void ipmi_fru_picmg_ext_print(uint8_t * fru_data, int off, int length)
 
 			for (i=0; i<entries; i++) {
 				printf("        HWAddr: 0x%02x  - SiteNum: 0x%02x - SiteType: 0x%02x \n",
-					fru_data[offset++], fru_data[offset++], fru_data[offset++]);
+					fru_data[offset], fru_data[offset+1], fru_data[offset+2]);
+				offset+= 3;
 			}
 		}
 		break;