Fix statseg v1 error offset zero 29/36729/1
authorNathan Skrzypczak <nathan.skrzypczak@gmail.com>
Mon, 25 Jul 2022 14:23:49 +0000 (16:23 +0200)
committerNathan Skrzypczak <nathan.skrzypczak@gmail.com>
Mon, 25 Jul 2022 14:23:53 +0000 (16:23 +0200)
This is the same fix as [0] for the statsseg v1 implementation
[0] https://gerrit.fd.io/r/c/govpp/+/36442

Signed-off-by: Nathan Skrzypczak <nathan.skrzypczak@gmail.com>
Change-Id: Ie22956152a83abe6371fe89482a66f2253b8493c

adapter/statsclient/statseg_v1.go

index 22052f5..3f15ce2 100644 (file)
@@ -102,10 +102,7 @@ func (ss *statSegmentV1) CopyEntryData(segment dirSegment, _ uint32) adapter.Sta
                return adapter.ScalarStat(dirEntry.unionData)
 
        case adapter.ErrorIndex:
-               if dirEntry.unionData == 0 {
-                       debugf("offset invalid for %s", dirEntry.name)
-                       break
-               } else if dirEntry.unionData >= uint64(len(ss.sharedHeader)) {
+               if dirEntry.unionData >= uint64(len(ss.sharedHeader)) {
                        debugf("offset out of range for %s", dirEntry.name)
                        break
                }