off by one buffer overflow

• 報告者: mortmann
• 担当者: cazfi
• チケットの種類: バグ
• 状況: 完了
• コンポーネント: General
• マイルストーン: 3.2.0
• 優先度: 5 - 中
• 重要度: 5 - 中
• 解決法: 修正済み
• ファイル: 1

詳細

here is an off by one overflow:

https://github.com/freeciv/freeciv/blob/afd94458c31a33f9d72d90c05c253a62f81cc4e0/common/unit.c#L585

see also:

https://github.com/freeciv/freeciv/blob/afd94458c31a33f9d72d90c05c253a62f81cc4e0/common/unit.c#L44

and the following statement is always TRUE and can therefore be removed:

https://github.com/freeciv/freeciv/blob/afd94458c31a33f9d72d90c05c253a62f81cc4e0/common/unit.c#L580

チケットの履歴 (8 件中 3 件表示)

2023-01-17 06:09 更新者: mortmann

• 新しいチケット "off by one buffer overflow" が作成されました

2023-01-17 06:21 更新者: cazfi

コメント

Reply To mortmann

and the following statement is always TRUE and can therefore be removed:

More than that, the entire real_activities array can be removed now that there's no non-real activities (holes) in the activities enum. I just didn't do it in at the same commit as removal of those non-real activities.

As for the reported bug, is it present in older branches too? (Likely so, don't think the code has changed)

2023-03-18 12:33 更新者: cazfi

コメント

Reply To cazfi

As for the reported bug, is it present in older branches too? (Likely so, don't think the code has changed)

No that I looked at it; yes and no. Older branches avoid actual over-boundaries write by luck. As they have also dummy activities included in the ACTIVITY_LAST count used in determining the size of the target array.

2023-03-18 12:33 更新者: cazfi

• マイルストーン が (未割り当て) から 3.0.7 (完了済み) に更新されました
• コンポーネント が (未割り当て) から General に更新されました

2023-03-18 12:45 更新者: cazfi

• マイルストーン が 3.0.7 (完了済み) から 3.2.0 に更新されました

2023-03-18 12:56 更新者: cazfi

• 添付ファイル 0012-Drop-deprecated-real_activities-array.patch (File ID: 11862) が付加されました

2023-03-18 12:56 更新者: cazfi

• 担当者 が (未割り当て) から cazfi に更新されました
• 解決法 が なし から 受領 に更新されました

2023-03-21 08:38 更新者: cazfi

• 状況 が オープン から 完了 に更新されました
• 解決法 が 受領 から 修正済み に更新されました