xtne6f / EDCB

BonDriver based multifunctional EPG software
Other
162 stars 37 forks source link

ユーザー名はOSの言語設定で変化するので、SIDを使用するように変更した #56

Closed Wakusei closed 2 years ago

Wakusei commented 2 years ago

NT AUTHORITY\Authenticated Usersというユーザー名、これがOSの言語設定によって変化することがあって、言語設定によっては失敗してしまいます。 英語と日本語はAuthenticated Usersですが、スペイン語だとUsuarios Autenticadosになります。 SIDで指定すると不変なので失敗しません。

日本語、英語の動作にももちろん支障はありません。 最小限の変更にしました。 いかがでしょうか?

xtne6f commented 2 years ago

ありがとうございます。ユーザー名も翻訳対象なんですね… 一点だけ、66行目の insecureFlag = true; はデバッグ時の名残でしょうか? ほかは問題ないように思います。

GrantAccessToKernelObjectWithSid()GrantAccessToKernelObject() に共通部分が多いのであとで統合しようと思います (些事ゆえこちらでやっておきます)。

Wakusei commented 2 years ago

デバッグ用のコードが残っていたので除去しました。 GrantAccessToKernelObjectWithSidは統一しておきました。

xtne6f commented 2 years ago

work-plus-s ブランチはリリース用のブランチなので、2つのコミットを misc ブランチに移動し、このプルリクエストの送り先を xtne6f:misc に変更することは可能でしょうか?(面倒でしたらこちらでcherry-pickします)

Wakusei commented 2 years ago

ブランチの説明は読んでましたが、リリース用だったのですね。 出しなおします。