NSData.dataWithBytesNoCopy_length_ is crashing

[liamappelbe]

ns_data_test.dart causes some sort of seg faults crash loop when using NSData.dataWithBytesNoCopy_length_. I'm switching to NSData.dataWithBytes_length_ for now, which fixes the crash but is slower. Logs.

I noticed the crash in #1100, but it predates that PR (that log is from a nightly run). I haven't been able to get GDB working on my macbook, so I'm not sure exactly where the crash is happening. From the logs it seems to be happening when the NSData is destroyed.

[brianquinlan]

@dcharkes

NSData dataWithBytesNoCopy:length:freeWhenDone: takes ownership of a block of memory that was allocated with malloc.

Is the package:ffi malloc the (presumably) libc malloc that NSData expects?

[dcharkes]

I haven't been able to get GDB working on my macbook

LLDB is the Mac variant.

Is the package:ffi malloc the (presumably) libc malloc that NSData expects?

native/pkgs/ffi/lib/src/allocation.dart

Lines 15 to 16 in 6a9282c

	final PosixMalloc posixMalloc =
	stdlib.lookupFunction<PosixMallocNative, PosixMalloc>('malloc');

Yep, it's malloc in the current process.

[brianquinlan]

I haven't been able to get GDB working on my macbook

LLDB is the Mac variant.

Is the package:ffi malloc the (presumably) libc malloc that NSData expects?

native/pkgs/ffi/lib/src/allocation.dart

Lines 15 to 16 in 6a9282c

	final PosixMalloc posixMalloc =
	stdlib.lookupFunction<PosixMallocNative, PosixMalloc>('malloc');

Yep, it's malloc in the current process.

I'm not sure how this fits together. Presumably the malloc of the current process could mean tcmalloc or jemalloc, depending on how the application is linked? But I'd guess that NSData uses whatever the standard deallocator is? Or maybe not ;-)

Apple's documentation isn't very clear on this. I also notice that we don't use NSData dataWithBytesNoCopy:length:freeWhenDone: at Google at all.

I think that this code is probably OK how it is now. If people complain about the performance when using large blocks of memory, we could use initWithBytesNoCopy:length:deallocator: