fpillet / NSLogger

A modern, flexible logging tool
Other
5k stars 573 forks source link

[CRASH] crash when opening .nsloggerrawdata in Desktop Viewer #304

Open quzongyao opened 3 years ago

quzongyao commented 3 years ago

macOS: Big Sur 11.1 Desktop Viewer: 1.9.7

Client nslogger: installed by cocoapods and version is 1.9.7

How to reproduce:

  1. open an nsloggerrawdata file recorded in my iOS app
  2. open the file in Desktop Viewer
  3. 💣

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.

Deub27 commented 3 years ago

Same issue here. It works with 1.9.0 version, not with version >= 1.9.5

nk-contorra commented 2 years ago

should be: NSData subset = [NSData dataWithBytesNoCopy:(unsigned char )p + 4 length:length freeWhenDone:NO];

aolan commented 2 years ago

Did you compile the package yourself?