チケット #47825

mapgen : check if there is at least one lake before regenerate_lakes

登録: 2023-04-10 03:17 最終更新: 2023-04-12 02:40

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

詳細

S3_1

with landmass85 , on a tiny map i had this error

3: Creating a map of size 30 x 40 = 1200 tiles (1000 requested).
../../../server/generator/mapgen_utils.c:349:37: runtime error: variable length array bound evaluates to non-positive value 0
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../../../server/generator/mapgen_utils.c:349:37 in 
Game saved as Sz1-Ai010-Lm85-230409_184441-T00001-auto.sav.xz

There i no lake on the map, and the error is about regenerate_lakes :-)

This did not prevented the map generation but it is better to fix it, see attached (trivial) patch for S3_1 , and also main i did not checked other branches, but i guess it will apply flawlessly.

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

2023-04-10 03:17 更新者: alain_bkr
  • 新しいチケット "mapgen : check if there is at least one lake before regenerate_lakes" が作成されました
2023-04-10 03:36 更新者: cazfi
コメント

If S2_6 is affected, there's no "world_object" in it. Would be nice to have this in 2.6.9 (in case you're not following freeciv-dev: we are about to release that in two weeks)

2023-04-10 07:38 更新者: alain_bkr
コメント

here is the patch for 2.6

(btw i subscribed to dev mailing list and read archive)

2023-04-10 11:16 更新者: cazfi
  • 担当者(未割り当て) から cazfi に更新されました
  • 解決法なし から 受領 に更新されました
  • マイルストーン(未割り当て) から 3.0.8 (完了済み) に更新されました
2023-04-12 02:40 更新者: cazfi
コメント

I think this was your first contribution with the commit having you as the author (as git knows it).

Once OpenHUB process freeciv data next time (frequency of that has fluctuated a lot during the years - sometimes it happens almost daily, but couple of months is normal too), it should show you as a new contributor, and if you wish, you can claim those contributions to your personal account.

https://www.openhub.net/p/freeciv/contributors/summary

2023-04-12 02:40 更新者: cazfi
  • 状況オープン から 完了 に更新されました
  • 解決法受領 から 修正済み に更新されました

添付ファイルリスト

編集

このチケットにコメントを追加するには、ログインが必要です » ログインする