コードの中身を見る限り、NSData
やNSMutableData
を多用していて、とてもSwift的に綺麗なコードとは言えないですね。著者に届かないところでケチをつけてもしかたないことなんで、これ以上は控えますが。
問題のエラーは解放済みの領域を再度解放(二重解放)しようとした時に表示されるものです。解放し過ぎなんですから、条件によってはbuffer.deallocate()
を削除することで解消できる可能性がありますが、別の条件ではご心配いただいているように領域の解放漏れでメモリリークが生じる可能性があります。
一番問題なのは、この行です。
AudioService.swift: 92行:
data = NSData(bytesNoCopy: buffer, length: Int(maxPacketCount * bytesPerPacket))
NSData.init(bytesNoCopy:length:)
と言うイニシャライザはbuffer
の所有権を引き継ぐ、つまり、buffer
の解放はNSData
の側が行うので、これを一度呼び出したら、buffer
を割り当てた側がその領域を解放する処理を呼び出してはいけません。
Declaration
init(bytesNoCopy bytes: UnsafeMutableRawPointer, length: Int)
Discussion
The returned object takes ownership of the bytes
pointer and frees it
on deallocation. Therefore, bytes
must point to a memory block
allocated with malloc
.
(拙訳)
返されたNSData
オブジェクトがbytes
ポインターの所有権を引き継ぎ、そのNSData
オブジェクトが廃棄(deallocation)される時にその領域をfree
で解放します。従って、bytes
はmalloc
で割り当てられたメモリ領域を指すものでないといけません。
元のコードは「malloc
で割り当てられたメモリ領域を指すものでないといけません」は守っているようですが、前半の事情がわかっていないようです。このイニシャライザを呼んだ後は、自前で同じ領域を解放すると冒頭に書いたような二重解放のエラー(場合によってはうまく検出されず、重大かつデバッグ困難な不具合となります)になります。
現象の説明が長くなってしまいましたが、先の行を次のように書き換えれば、buffer
についての 二重解放の問題はなくなります。
data = NSData(bytesNoCopy: buffer, length: Int(maxPacketCount * bytesPerPacket), freeWhenDone: false)
freeWhenDone: false
と言う引数を追加することにより、所有権はNSData
の側に移らず、引き続きbuffer
を割り当てた側がその領域を解放する責任を持つことになります。
ただ、イニシャライザとデイニシャライザが全然釣り合っていませんね。
init(_ obj: Any?) {
startingPacketCount = 0
maxPacketCount = (48000 * seconds)
buffer = UnsafeMutableRawPointer(malloc(Int(maxPacketCount * bytesPerPacket)))
}
deinit {
buffer.deallocate()
}
buffer
はmalloc
で割り当てられているのですから、解放はfree
で行うべきです。(現在のSwiftの実装ではdeallocate()
は内部でfree
を呼んでいるだけなので、動くことは動くでしょうが。)
また、そもそもSwiftのmalloc
の戻り値型はUnsafeMutableRawPointer!
型になるので、型変換のためにUnsafeMutableRawPointer(
)
で囲んでやる必要もありません。
なんで使いもしていないobj
なんて引数があるのか、なんてことも解せないんですが、とりあえずそれは置いといて、以下のように描き直すべきでしょう。
init(_ obj: Any?) {
startingPacketCount = 0
maxPacketCount = (48000 * seconds)
buffer = malloc(Int(maxPacketCount * bytesPerPacket))
}
deinit {
free(buffer)
}
ついでに47行、buffer
は一度割り当てられたら気安く書き換えられては困るのでvar
ではなくlet
の方が良いですね。(他にもlet
にした方が良いものはまだありそうですが。)
let buffer: UnsafeMutableRawPointer
まだまだ、メモリ絡みで何か出てきそうな気もしますが、これ以上は本題から外れすぎるように思うので、深追いはせずに置きます。
上記の修正を行えば、二重解放でのエラーは起こらなくなるはずなのでお試しください。