質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
iOS

iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

Q&A

解決済

2回答

2851閲覧

SecTrustがリークする

MetroHighSpeed

総合スコア7

iOS

iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

0グッド

0クリップ

投稿2017/03/30 06:02

###前提・実現したいこと
Securityフレームワークを使用し、X.509形式の証明書を検証する処理を実装しました。
検証処理自体は正常に完了するのですが、その際にメモリリークが発生しているようです。
処理を抜ける前に適切に解放したいのですが、どのようにすればいいのかわかりません。

###発生している問題・エラーメッセージ
処理を1回実行した直後をDebug Memory Graphで確認したものです。

グラフを見るとSecTrustがどこからも参照されておらず、他のSec〜オブジェクトはSecTrustから参照されていて解放されていないため、SecTrustを解放できれば問題が解決するのではと考えています。

###該当のソースコード
certDataはCFData型の証明書データです。
trustPtrSecTrust?のポインタなので、その領域をリリースしてもさらにそのオプショナル型から参照されている(?この辺いまいち内部の仕組みがわかっていません)SecTrustのオブジェクトは解放されていないのではないのか、と思うのですが、どうすれば解放されるのかがわかりません。

Swift

1let trustPtr = UnsafeMutablePointer<SecTrust?>.allocate(capacity: 1) 2let resultPtr = UnsafeMutablePointer<SecTrustResultType>.allocate(capacity: 1) 3 4let cert = SecCertificateCreateWithData(kCFAllocatorDefault, certData) 5SecTrustCreateWithCertificates(cert!, SecPolicyCreateBasicX509(), trustPtr) 6SecTrustEvaluate(trustPtr.pointee!, resultPtr) 7if(resultPtr.pointee != SecTrustResultType.unspecified && resultPtr.pointee != SecTrustResultType.proceed) { 8 print("Invalid certificate") 9} 10 11trustPtr.deallocate(capacity: 1) 12resultPtr.deallocate(capacity: 1)

###補足情報(言語/FW/ツール等のバージョンなど)
Swift 3.0.2
Xcode 8.2.1
iOS 10.2

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

Unmanagedを使用し、SecTrustオブジェクトの参照を得て、それを解放することができました。

Swift

1let trustPtr = UnsafeMutablePointer<SecTrust?>.allocate(capacity: 1) 2let resultPtr = UnsafeMutablePointer<SecTrustResultType>.allocate(capacity: 1) 3 4let cert = SecCertificateCreateWithData(kCFAllocatorDefault, certData) 5SecTrustCreateWithCertificates(cert!, SecPolicyCreateBasicX509(), trustPtr) 6SecTrustEvaluate(trustPtr.pointee!, resultPtr) 7if(resultPtr.pointee != SecTrustResultType.unspecified && resultPtr.pointee != SecTrustResultType.proceed) { 8 print("Invalid certificate") 9} 10Unmanaged.passUnretained(trustPtr.pointee!).release() //この行を追加 11trustPtr.deallocate(capacity: 1) 12resultPtr.deallocate(capacity: 1)

参照を増やす必要はないのでpass​Unretainedでよいかと思います。

投稿2017/03/31 02:04

MetroHighSpeed

総合スコア7

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

SecCertificate、SecTrustについては、

Call the CFRelease function to release this object when you are finished with it.

と、ドキュメントに書かれています。

Sec​Certificate​Create​With​Data(:​:​)
Sec​Trust​Create​With​Certificates(:​:​_:​)

UnsafeMutablePointerを使わない

Swift 2.3 (ビルドだけ確認、動作未確認)

swift

1var trust: SecTrustRef? 2var result: SecTrustResultType = SecTrustResultType(kSecTrustResultInvalid) 3 4let cert = SecCertificateCreateWithData(kCFAllocatorDefault, certData) 5SecTrustCreateWithCertificates(cert!, SecPolicyCreateBasicX509(), &trust) 6SecTrustEvaluate(trust!, &result) 7if (result != SecTrustResultType(kSecTrustResultUnspecified) && result != SecTrustResultType(kSecTrustResultProceed)) { 8 print("Invalid certificate") 9}

Swift 3.x (ビルドも動作も大丈夫らしい)

swift

1var trust: SecTrust? 2var result: SecTrustResultType = .invalid 3 4let cert = SecCertificateCreateWithData(kCFAllocatorDefault, certData) 5SecTrustCreateWithCertificates(cert!, SecPolicyCreateBasicX509(), &trust) 6SecTrustEvaluate(trust!, &result) 7if (result != .unspecified && result != .proceed) { 8 print("Invalid certificate") 9}

投稿2017/03/30 06:35

編集2017/03/31 02:14
fuzzball

総合スコア16731

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

MetroHighSpeed

2017/03/30 06:43

CFReleaseはSwift3で利用できるのですか?私の環境では 'CFRelease' is unavailable: Core Foundation objects are automatically memory managed と言われコンパイルできないのですが。
fuzzball

2017/03/30 06:49

私はSwift3環境ではないので確認できません。
MetroHighSpeed

2017/03/30 06:57

発生している現象も自身の回答も確認する方法がないのにこの回答をしたんですね。それはちょっとひどい・・・ Teratailって低評価って押せないんですね。
fuzzball

2017/03/30 07:03

なんだこいつはw
fuzzball

2017/03/30 07:05

「低評価」って押せないですか?質問者は押せないのかな?
fuzzball

2017/03/30 07:06

発生している現象も自身の回答も確認する方法もなくても、問題を解決できることはあります。
MetroHighSpeed

2017/03/30 07:11

すいません。せっかく回答をいただいたのに。もちろん確認方法なんてなくてもその方が熟知しているようなことであれば正しい回答はできると思います。 ですが、私が伝えたいことは「この回答では解決しない」ということです。 低評価は私からはとてもうすいグレーになっていて押せないようになっています。ベストアンサーと高評価しか選択できないように見えます。
fuzzball

2017/03/30 07:18

グレーになっているのは、無闇に押さないようにするためだと思いますので、押してみて下さい。
fuzzball

2017/03/31 00:47

UnsafeMutablePointerを使わない方法を(妄想で)書いてみました。
MetroHighSpeed

2017/03/31 01:57

このままではSwift3ではビルドできませんが、回答のコードのSecTrustRef?をSecTrust?(Swift3から名前が変更になった模様)にすることで、正常に動作し、Sec〜が全て解放されリークが発生しないことが確認できました! ある型のUnsafeMutablePointerを要求している箇所はinoutパラメータと捉えることができ、'&'をつけてその型のinout値を渡すことで、Swiftのメモリマネージ対象になるということですかね?その辺を説明しているドキュメントが見つからないのでなんともいえませんが…。 私の方でも別の方法を見つけたのですが(別回答を参照)、こちらの方がコードが少なくまた安全であるのではないかと思います。
fuzzball

2017/03/31 02:13

色々ググっていたら、みんな普通に(というかC言語っぽく)&付けていたので真似してみただけです。難しいことは分かりません。 回答は修正しておきました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問