Skip invalid WPILOG control records

Mechanical-Advantage · Oct 8, 2024 · df8f36e · df8f36e
1 parent a103fc4
commit df8f36e
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/hub/dataSources/wpilog/wpilogWorker.ts b/src/hub/dataSources/wpilog/wpilogWorker.ts
@@ -47,8 +47,10 @@ async function start(data: Uint8Array) {
       },
       (entry, position) => {
         if (entry === CONTROL_ENTRY) {
-          let record = decoder!.getRecordAtPosition(position)[0]!;
-          if (record.isStart()) {
+          let record = decoder?.getRecordAtPosition(position)[0];
+          if (record === null || record === undefined) {
+            console.warn("Encountered invalid control record at offset", position);
+          } else if (record.isStart()) {
             const startData = record.getStartData();
             entryIds[startData.entry] = startData.name;
             entryTypes[startData.name] = startData.type;
@@ -98,7 +100,7 @@ async function start(data: Uint8Array) {
               log.setMetadataString(entryIds[setMetadataData.entry], setMetadataData.metadata);
             }
           }
-        } else {
+        } else if (entry in entryIds) {
           let key = entryIds[entry];
           dataRecordPositions[key].push(position);
         }