Skip to content

Commit

Permalink
Added missing check for a nil error.
Browse files Browse the repository at this point in the history
  • Loading branch information
pmarkowsky committed Sep 22, 2021
1 parent f0a81ff commit aa59cb1
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions Source/santametricservice/Formats/SNTMetricRawJSONFormat.m
Original file line number Diff line number Diff line change
Expand Up @@ -79,13 +79,14 @@ - (NSDictionary *)normalize:(NSDictionary *)metrics {
NSDictionary *normalizedMetrics = [self normalize:metrics];

if (![NSJSONSerialization isValidJSONObject:normalizedMetrics]) {
*err = [[NSError alloc]
initWithDomain:@"SNTMetricRawJSONFileWriter"
code:EINVAL
userInfo:@{
NSLocalizedDescriptionKey : @"unable to convert metrics to JSON: invalid metrics"
}];

if (err != nil) {
*err = [[NSError alloc]
initWithDomain:@"SNTMetricRawJSONFileWriter"
code:EINVAL
userInfo:@{
NSLocalizedDescriptionKey : @"unable to convert metrics to JSON: invalid metrics"
}];
}
return nil;
}

Expand Down

0 comments on commit aa59cb1

Please sign in to comment.