0

https://github.com/google/macops-keychainminder/blob/b6ab824321fc677aa80a217348023dcdf416b68a/Common/Common.m

https://github.com/bikram990/NoMADLogin-AD/blob/a8ff99b12335bbaa307052b4df27ca3163b8f663/Carthage/Checkouts/NoMAD-ADAuth/NoMAD-ADAuth/KerbUtil.m
に記載されている AuthorizationEnvironmentを生成するObjective-CのコードをSwiftで書くにはどうすればよいでしょうか?

自分なりに、下記のように書きました

var authorizationRef: AuthorizationRef?
let authorizationFlags: AuthorizationFlags = [
    .extendRights,
    .interactionAllowed,
    .preAuthorize
]

var userName = NSUserName()
var password = "password"

let userNameItem = AuthorizationItem(
    name: kAuthorizationEnvironmentUsername,
    valueLength: userName.lengthOfBytes(using: .utf8),
    value: UnsafeMutableRawPointer(&userName),
    flags: 0
)
let passwordItem = AuthorizationItem(
    name: kAuthorizationEnvironmentPassword,
    valueLength: password.lengthOfBytes(using: .utf8),
    value: UnsafeMutableRawPointer(&password),
    flags: 0
)
var items = [
    userNameItem,
    passwordItem
]

var authorizationEnvironment = AuthorizationEnvironment(
    count: 2,
    items: &items
)

let status = AuthorizationCreate(
    nil,
    &authorizationEnvironment,
    authorizationFlags,
    &authorizationRef
)

以下略

ところが下記の警告が出ています。

Passing 'String' to parameter, but argument 'name' should be a pointer that outlives the call to 'init(name:valueLength:value:flags:)'

Inout expression creates a temporary pointer, but argument 'items' should be a pointer that outlives the call to 'init(count:items:)'

Initialization of 'UnsafeMutableRawPointer' results in a dangling pointer

といったものです。

また作り出したAuthorizationEnvironmentインスタンスAuthorizationCreateに渡した結果、Objective-Cだとパスワードの要求をされなくなったのですが、Swiftで同様のコードを再現しようとすると、パスワードを要求されるので、おそらくAuthorizationEnvironmentインスタンスの生成がうまくできていないのだと推測しています。

蛇足かもしれませんが、はじめ userName変数はletで宣言していたのですが、

Cannot pass immutable value as inout argument: 'userName' is a 'let' constant

というコンパイルエラーが出たので意味もわからず var で宣言しなおして、コンパイルエラーを解消しています。(その他 password変数も同様の理由で var で宣言しています)

1 件の回答 1

1

Passing 'String' to parameter, but argument 'name' should be a pointer that outlives the call to 'init(name:valueLength:value:flags:)'

Swiftでは、String型の値をUnsafePointer<Int8>型の引数に渡すことができますが、その時に生成されるポインタは一時的なもので、「渡されたポインタを後で使うために保持する」ような構造体を作成する場合には、そのまま使用してはいけません。この使い方は残念ながらある特定の条件が揃ってしまうと動いているように見えることがあるため、例えば「本番環境でだけ現れる厄介なエラー」なんかの原因になったりすることがあります。

できるだけ安全にそのようなポインタを作りたければ、書き換え可能な領域にメモリを割り当ててコピーを保持する必要があるでしょう。C標準関数のstrdupなんかが使えます。

Inout expression creates a temporary pointer, but argument 'items' should be a pointer that outlives the call to 'init(count:items:)'

配列をポインタ型の変数に渡すときにも同様の「一時的なポインタ」が作られるので、上記と同じことが言えます。文字列と同様に安定な領域にメモリを割り当ててコピーを作るか、この場合ならポインタが使用される可能性のある範囲をwithUnsafeMutableBufferPointerなどで囲んで使ってやることになります。

Initialization of 'UnsafeMutableRawPointer' results in a dangling pointer

UnsafeMutableRawPointer(&userName)のような使い方は非常によく見られる(そこそこ有名なライブラリでも使われていたことがあると記憶しています)誤った使い方の典型例です。UnsafeMutableRawPointer.initに文字列を指定すると、一時的な領域が作られ、その領域のアドレスが渡されるのですが、その領域のアドレスはUnsafeMutableRawPointer.initが終了すると、無効になります。

先ほども書きましたが、この「誤った使い方」の厄介なところは、特定の状況ではそれが何度繰り返して実行しても動いているように見えてしまうことです。

Xcodeのポインタに関する警告は絶対に無視しないようにしましょう。

  • テスト環境では動くのに、本番環境では動かない
  • 特定の(多くの場合「以前の」)バージョンのiOSでは動くのに、別のバージョン(例えば「最新の」)では動かない

なんてことが起こると、「これはiOSのバグのせいに違いない」なんて書き込みをされる方がAppleのDevForumsなんかに時々現れるのですが、コードを見せてもらうと「ほら、こんなところで間違ったポインタの使い方をしています」なんてことになります。
(誤ったマルチスレッド処理でも同じような状況が発生しますが。)

Cannot pass immutable value as inout argument: 'userName' is a 'let' constant

こういったC言語ベースのAPIしか用意されていない場合、絶対に内容を書き換えないだろうポインタにconst等の宣言がなされていないことがよくあって、その場合、mutableなポインタが必要になります。そのためにはその元になる変数をvarにする必要が出てくる、と言うわけで、この場合、「意味もわからず var で宣言しなおし」なんてことも必要になるでしょう。

先に書いた「一時的ポインタをあとあと使用される場所に使おうとしている」のさえ気をつければ、そこらへんは「とにかく var にする」と言う対応にならざるを得ません。


と言うわけでざっくりと上記の状況を解決すると、以下のようなコードになります。

    var authorizationRef: AuthorizationRef?
    let authorizationFlags: AuthorizationFlags = [
        .extendRights,
        .interactionAllowed,
        .preAuthorize
    ]
    
    let userName = NSUserName()
    let password = "password"
    
    let cAuthorizationEnvironmentUsername = strdup(kAuthorizationEnvironmentUsername)!
    defer {free(cAuthorizationEnvironmentUsername)}
    let cUserName = strdup(userName)!
    defer {free(cUserName)}
    
    let cAuthorizationEnvironmentPassword = strdup(kAuthorizationEnvironmentPassword)!
    defer {free(kAuthorizationEnvironmentPassword)}
    let cPassword = strdup(password)!
    defer {free(cPassword)}
    
    let userNameItem = AuthorizationItem(
        name: cAuthorizationEnvironmentUsername,
        valueLength: userName.utf8.count,
        value: cUserName,
        flags: 0
    )
    
    let passwordItem = AuthorizationItem(
        name: cAuthorizationEnvironmentPassword,
        valueLength: password.utf8.count,
        value: cPassword,
        flags: 0
    )
    var items = [
        userNameItem,
        passwordItem
    ]
    items.withUnsafeMutableBufferPointer {bufItems in
        var authorizationEnvironment = AuthorizationEnvironment(
            count: UInt32(bufItems.count),
            items: bufItems.baseAddress
        )
        let status =  AuthorizationCreate(
            nil,
            &authorizationEnvironment,
            authorizationFlags,
            &authorizationRef
        )
        //`authorizationRef`はこのブロック内で使う必要がある?
        //...
    }

Swiftのポインタに関する警告を安全に黙らせることだけを考えて書いたので、実際に使う分には何か問題が出るかもしれません。その場合にはコメント等でお知らせください。

4
  • ありがとうございます。これはすごいですね。 defer {free(cAuthorizationEnvironmentUsername)} malloc: *** error for object 0x600002cd83a0: pointer being freed was not allocated が発生したので、freeの箇所をコメントアウトする形で使用すると意図通り動きました。(このままコメントアウトのまま使うのはおそらくよくないと思いますが) Commented 2021年1月26日 6:20
  • @shingo.nakanishi さん、コメントありがとうございます。defer {free(cAuthorizationEnvironmentUsername)}が2つになってますが、片方はPasswordでないといけないですね。修正しときますので、確認してみてください。
    – OOPer
    Commented 2021年1月26日 6:28
  • @OOPerさん。たしかに、もしかしたら、私のはじめの質問で両方Usernameになっていたなごりかもしれません(初期の段階で質問修正済みですが)。片方Passwordに修正するとエラーなくなりました。 Commented 2021年1月26日 6:34
  • 1
    @shingo.nakanishi さん、その部分は気づいて直したはずなのに defer を書くときにコピペした後修正し忘れ、ってやつです。ともあれエラーが出ずに動くようになったのであれば幸いです。
    – OOPer
    Commented 2021年1月26日 6:38

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。