Open TwoSquirrels opened 2 months ago
まず部屋数が 1000000 を超えるとここで無限ループする上、近づくだけでもかなり時間がかかると予想され危険です。
https://github.com/traP-jp/ShoutWars-server/blob/1a92cde392e0965e7b8cdffab6cbfde00dec28c5/room_list.cpp#L26-L28
そもそも部屋番号が推測できるべきではないため、部屋数上限の設定可能範囲を 1~500000 ぐらいにして実行時チェックさせるのがよさそう。1000 超えてたら警告出すとか
優先順位はめちゃ低なので、とりあえず放置で
まず部屋数が 1000000 を超えるとここで無限ループする上、近づくだけでもかなり時間がかかると予想され危険です。
https://github.com/traP-jp/ShoutWars-server/blob/1a92cde392e0965e7b8cdffab6cbfde00dec28c5/room_list.cpp#L26-L28
そもそも部屋番号が推測できるべきではないため、部屋数上限の設定可能範囲を 1~500000 ぐらいにして実行時チェックさせるのがよさそう。1000 超えてたら警告出すとか