From ffe7ccbf3bdc03339e9d6774f588aed268c07b78 Mon Sep 17 00:00:00 2001
From: zhaoertao <ertaozh@cn.ibm.com>
Date: Thu, 24 Oct 2013 02:09:20 -0700
Subject: [PATCH] fix bug 3820: rinv for feihu FPC is producing errors

---
 xCAT-server/lib/xcat/plugins/ipmi.pm | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/xCAT-server/lib/xcat/plugins/ipmi.pm b/xCAT-server/lib/xcat/plugins/ipmi.pm
index 30d849551..f40ef3c27 100644
--- a/xCAT-server/lib/xcat/plugins/ipmi.pm
+++ b/xCAT-server/lib/xcat/plugins/ipmi.pm
@@ -2911,8 +2911,10 @@ sub parsefru {
             return "unknown-COULDGUESS",undef; #be lazy for now, TODO revisit this and add guessing if it ever matters
         }
         $currsize=($bytes->[$curridx+1])*8;
-        @currarea=@{$bytes}[$curridx..($curridx+$currsize-1)]; #splice @$bytes,$curridx,$currsize;
-        $fruhash->{chassis} = parsechassis(@currarea);
+        if ($currsize > 0) {
+            @currarea=@{$bytes}[$curridx..($curridx+$currsize-1)]; #splice @$bytes,$curridx,$currsize;
+            $fruhash->{chassis} = parsechassis(@currarea);
+        }
     }
     if ($bytes->[3]) { #Board info area, to be preserved
         $curridx=$bytes->[3]*8;
@@ -2920,8 +2922,10 @@ sub parsefru {
             return "unknown-COULDGUESS",undef;
         }
         $currsize=($bytes->[$curridx+1])*8;
-        @currarea=@{$bytes}[$curridx..($curridx+$currsize-1)];
-        $fruhash->{board} = parseboard(@currarea);
+        if ($currsize > 0) {
+            @currarea=@{$bytes}[$curridx..($curridx+$currsize-1)];
+            $fruhash->{board} = parseboard(@currarea);
+        }
     }
     if (ref $global_sessdata->{currmacs}) {
 	$fruhash->{board}->{macaddrs}=[];
@@ -2938,8 +2942,10 @@ sub parsefru {
             return "unknown-COULDGUESS",undef;
         }
         $currsize=($bytes->[$curridx+1])*8;
-        @currarea=@{$bytes}[$curridx..($curridx+$currsize-1)];
-        $fruhash->{product} = parseprod(@currarea);
+        if ($currsize > 0) {
+            @currarea=@{$bytes}[$curridx..($curridx+$currsize-1)];
+            $fruhash->{product} = parseprod(@currarea);
+        }
     }
     if ($bytes->[5]) { #Generic multirecord present..
         $fruhash->{extra}=[];