[CRASH] crash when opening .nsloggerrawdata in Desktop Viewer
quzongyao opened this issue · comments
macOS: Big Sur 11.1
Desktop Viewer: 1.9.7
Client nslogger: installed by cocoapods and version is 1.9.7
How to reproduce:
- open an nsloggerrawdata file recorded in my iOS app
- open the file in Desktop Viewer
- 💣
Reproducing Rate:
100%
I also tried to build the Desktop Viewer myself from the master branch and crash is occurring.
And after the following change, no crash any more.
in LoggerDocument.m
- (BOOL)readFromData:(NSData *)data ofType:(NSString *)typeName error:(NSError **)outError {
LINE 312
- NSData *subset = [NSData dataWithBytesNoCopy:(unsigned char *)p + 4 length:length];
+ NSData *subset = [NSData dataWithBytes:(unsigned char *)p + 4 length:length];
I checked the code, p is from the input param, and dataWithBytesNoCopy will hand the ownership from to subset.
This could cause the input param data to be deallocated twice.
Please check my analysis, thank you.
Sorry I don't have much time to make a PR.
Same issue here.
It works with 1.9.0 version, not with version >= 1.9.5
should be:
NSData *subset = [NSData dataWithBytesNoCopy:(unsigned char *)p + 4 length:length freeWhenDone:NO];
Did you compile the package yourself?