Adds error handling for serialization failure in MPDB readRows

mixpanel · Jan 29, 2025 · 4256dfe · 4256dfe
1 parent 0b982aa
commit 4256dfe
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/Sources/MPDB.m b/Sources/MPDB.m
@@ -210,6 +210,11 @@ - (NSArray *)readRows:(NSString *)persistenceType numRows:(NSInteger)numRows fla
                     int eId = sqlite3_column_int(selectStatement, 0);
                     NSError *error;
                     NSMutableDictionary *jsonObject = [NSJSONSerialization JSONObjectWithData:blob options:NSJSONReadingMutableContainers error:&error];
+                    if (error) {
+                        NSString* utf8BlobString = [[NSString alloc] initWithData:blob encoding:NSUTF8StringEncoding];
+                        [self logSqlError:[NSString stringWithFormat:@"Serialization error in %@ for column blob %@", tableName, utf8BlobString]];
+                        continue;
+                    }
                     jsonObject[@"id"] = [NSNumber numberWithInt:eId];
                     [rows addObject:jsonObject];
                     rowsRead++;