セキュリティの関わる問題なので削除させていただきました。

Question

セキュリティの関わる問題なので削除させていただきました。

Answer 1

SOSを頂きましたが、amaguriさん自身で答えを述べていますよね。
あとは、
CommonComponent::CheckMatchDB()
CommonComponent::CheckMatchQuery()
をそれぞれ確認するだけです。

ちなみに

$check_cnt++; //$check_cntを１にする。

これは違います。
PHP: 加算子/減算子 - Manual

Comments

[amaguri]

for文のループの詳細がイマイチつかめず。。
なんともというかんじです。。
$check_cnt++;の件
$check_cnt=0;
$check_cnt++;
とありましたが
$check_cnt=１;
につすというのはまちがいだったでしょうか？？

[coba-coba]

> for文のループの詳細がイマイチつかめず。。
> なんともというかんじです。。

すみません、何が分からないのかがよく分かりません。
もう少し具体的に何が引っかかっているのか教えてもらえますか。

> $check_cnt++;の件
> $check_cnt=0;
> $check_cnt++;
> とありましたが
> $check_cnt=１;
> につすというのはまちがいだったでしょうか？？

またまた申し訳ないですが……、何を言っているか分かりません。笑
「$check_cnt++;」が示す意味は「$check_cnt=１;」と同じであるか、という意味でしょうか。

改めてコードを見たら
$valid_cnt++;//$valid_cnt+1
とコメントしている行もありますね。
問題なかったかもしれません。

[amaguri]

for ( $i=0; $i<count($results); $i++){}
の文で
for文
式１の$u_type=$results[$i]が０と時っていうのが$u_type＝無効->type;となる時の意味がよくわからず

式２の$i<count($results)も$resultsのオブジェクトを数えて何しているのか？

式３の$u_type=$results[$i]が＋１されて
が$u_type＝１->type;

となるのがそもそも$results[$i]が$results[０]や$results[１]になったとき
これは$results[０]が無効の時　$results[１]が有効の時なのか？
そもそもこれは何のためのループなのか
その辺踏まえてよくわからないです。。

[coba-coba]

今回のamaguriさん自身のコメントを一度読み直して、修正してもらえますか？
意地悪で言うわけではないのですが、何をおっしゃっているのか分かりません…。

【分からない点】
１．
式１・式２・式３とはどれを指すのか（複数あって、どこのことを言っているのか分からない）

２．
「式１の$u_type=$results[$i]が０と時っていうのが$u_type＝無効->type;となる時の意味がよくわからず」の部分

３．
「式３の$u_type=$results[$i]が＋１されてが$u_type＝１->type;となるのがそもそも$results[$i]が$results[０]や$results[１]になったときこれは$results[０]が無効の時　$results[１]が有効の時なのか？」の部分

[amaguri]

for ( $i=0; $i<count($results); $i++){のfor文の中で

$i=0;の時
$u_type=$results[$i]->type;
$u_db=$results[$i]->db;

$i<count($results); の時
$u_type=$results[$i]->type;
$u_db=$results[$i]->db;

$i++;
$u_type=$results[$i]->type;
$u_db=$results[$i]->db;

の時の動作が詳細な動きが理解できていません。

また、なぜループをしているかの理由もわからないです

[amaguri]

if ( substr($chk_ptn,0,1) == '!' ){
の '!'の意味もよくわからないです
!は否定だと思うのですが。。
空白ではないよ
という意味なら
if ( !substr($chk_ptn,0,1) == '' ){
にすると思いますし、、

[coba-coba]

やっていることは次のようなことです。

１．ユーザーと権限がセットになったリストを取得
２．EXPLAINファイルから取得したデータベース名とクエリ（クエリタイプ）を$resultsから1つずつ取り出す。
３．２のデータベース名とクエリが１のリストとマッチするかを調べ、マッチした数だけ$valid_cntに1を足す。（CommonComponent::CheckMatchDB()とCommonComponent::CheckMatchQuery()で何かを調べているが、何を調べているか不明なので、もしかしたらマッチしない数を$valid_cntで数えている可能性もあり。）
４．２と３の手順を$resultsの持つデータの数だけ繰り返す。（count($results)の分だけ繰り返す。）

この流れを考えながらコードを見直してください。

[coba-coba]

> if ( substr($chk_ptn,0,1) == '!' ){
> の '!'の意味もよくわからないです
> !は否定だと思うのですが。。

substr($chk_ptn,0,1)が返す文字列と文字列としての'!'が一致するかを比較しています。

[amaguri]

なるほど
$resultsには複数データが入っている可能性があるのでそれを全部一個ずつ取り出して見ているということでしょうか！？

[coba-coba]

そうです。

Answer 2

ループの中の判定を外だししてみました。
少しは見やすくなったかな?

        if (count($users) == 0) {
            for ($i=0; $i<count($results); $i++) {
                $check_cnt++;
                $u_type = $results[$i]->type;
                $u_db   = $results[$i]->db;
                if (CommonComponent::CheckMatchDB(DATABASE_PERMISSION, $u_db) != 0) {
                    continue;
                }
                if (CommonComponent::CheckMatchQuery($u_type, LDAP_AUTH) != 0) {
                    return 1;
                }
                $valid_cnt++;
            }
        } else {
            for ($i=0; $i<count($results); $i++) {
                $check_cnt++;
                $u_type = $results[$i]->type;
                $u_db   = $results[$i]->db;
                for ($j=0; $j<count($users); $j++) {
                    $u_auth = $users[$j]['Roles']['authority'];
                    $u_ptn  = $users[$j]['Roles']['hive_database'];
                    if (CommonComponent::CheckMatchDB($u_ptn, $u_db) != 0) {
                      continue;
                    }
                    if (CommonComponent::CheckMatchQuery($u_type, $u_auth) != 0) {
                      return 1;
                    }
                    break;
                }
                $valid_cnt++;
            }
        }

// 最初の if文がうまくインデントできないなあ。

Comments

[amaguri]

お返事ありがとうございます。
解析していたのをかいてしまっていたのでごちゃごちゃしていましたよねすみません」