バグを修正 #1

suwako · 2023-12-05T17:53:43+09:00

suwako がコメント

2023-12-05 17:53:43 +09:00

貢献者

説明：

文字列の場合、「<=」や「>」を使ったら、バグが発生する可能性があるから、「==」か「!=」を使った方が良い
「@」を使うと、問題があれば、エクスプロイトを起こる可能性があるので、エラーチェックを作った方が良い
「$var = null;」＝「$varはまだ存在するけど、NULLに設定する事」、「unset($var);」＝「$varがメモリから削除する」
「chmod()」はほとんど必要はないから、消した
ファイルやディレクトリが存在かどうかチェックを追加した
mkdirは失敗したら、falseを戻す。でも、美々さんはこのケースを全然チェックしないから、mkdirに失敗すると、バグが発生する
「if ($size <= 0)　→　if ($size > 1024 * 500)」＝「$size <= 0」はtrueだったら、「$size > 1024 * 500」もtrueかどうか確認しよう。「if ($size <= 0)　→　else if ($size > 1024 * 500)」＝「$size <= 0」はtrueだったら、「$size > 1024 * 500」を確認する事が必要はない

説明： * 文字列の場合、「<=」や「>」を使ったら、バグが発生する可能性があるから、「==」か「!=」を使った方が良い * 「@」を使うと、問題があれば、エクスプロイトを起こる可能性があるので、エラーチェックを作った方が良い * 「$var = null;」＝「$varはまだ存在するけど、NULLに設定する事」、「unset($var);」＝「$varがメモリから削除する」 * 「chmod()」はほとんど必要はないから、消した * ファイルやディレクトリが存在かどうかチェックを追加した * mkdirは失敗したら、falseを戻す。でも、美々さんはこのケースを全然チェックしないから、mkdirに失敗すると、バグが発生する * 「if ($size <= 0)　→　if ($size > 1024 * 500)」＝「$size <= 0」はtrueだったら、「$size > 1024 * 500」もtrueかどうか確認しよう。「if ($size <= 0)　→　else if ($size > 1024 * 500)」＝「$size <= 0」はtrueだったら、「$size > 1024 * 500」を確認する事が必要はない

suwako が 1 コミット追加 2023-12-05 17:53:46 +09:00

f1955044e4 バグを修正

tak4 がコミット 24a7fc87ed を master にマージ 2023-12-05 23:02:40 +09:00

tak4 がコミットでこのイシューを参照

2023-12-05 23:02:40 +09:00

Merge pull request 'バグを修正' (#1) from suwako/bibis:master into master

tak4 がコメント

2023-12-05 23:11:18 +09:00

オーナー

有難うございます。簡単に確認して問題無かったのでマージしました。

じつはTor上のBIBISはmasterより少し新しいです。先ほどpushしました……衝突があるので時間のあるときにこれもマージしておきます。

https://gitler.moe/tak4/bibis/src/branch/develop

有難うございます。簡単に確認して問題無かったのでマージしました。じつはTor上のBIBISはmasterより少し新しいです。先ほどpushしました……衝突があるので時間のあるときにこれもマージしておきます。 https://gitler.moe/tak4/bibis/src/branch/develop

サインインしてこの会話に参加。

レビューアなし

ラベルなし

マイルストーンなし

プロジェクトなし

担当者なし

2 人の参加者

通知

期日

期日が正しくないか範囲を超えています。 'yyyy-mm-dd' の形式で入力してください。

期日は未設定です。

依存関係

依存関係が設定されていません。

リファレンス: tak4/bibis#1