チケット #43871

luascript_func_check(): variable used uninitialized on FREECIV_NDEBUG build

登録: 2022-02-15 20:00 最終更新: 2022-03-06 02:36

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

詳細

luascript_func_check() sets variable pfunc within fc_assert_ret_val(), i.e., it's not set in FREECIV_NDEBUG builds. Later code expects it to be set.

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

2022-02-15 20:00 更新者: cazfi
  • 新しいチケット "luascript_func_check(): variable used uninitialized on FREECIV_NDEBUG build" が作成されました
2022-02-20 23:26 更新者: cazfi
  • 担当者(未割り当て) から cazfi に更新されました
  • 解決法なし から 受領 に更新されました
  • マイルストーン(未割り当て) から 3.0.1 (完了済み) に更新されました
コメント

Will push also to S2_6.

2022-03-04 04:14 更新者: cazfi
  • 解決法受領 から なし に更新されました
コメント

As fc_assert_ret_val() is one of the macros of the fc_assert family that we still have active even on --disable-debug builds, there's no *currently* functional bug, and this patch causes compilation failure instead.

Even in case of fc_assert_ret_val(), it's forbidden to have functionality inside it, for the purposes of being able to disable them on ndebug builds on some future versions. So this patch is still needed, just have to fix the compile error.

2022-03-04 04:23 更新者: cazfi
  • 解決法なし から 受領 に更新されました
コメント

Reply To cazfi

As fc_assert_ret_val() is one of the macros of the fc_assert family that we still have active even on --disable-debug builds, there's no *currently* functional bug, and this patch causes compilation failure instead.

No, that was true for S2_6 only. In later branches the functionality is broken + there's no compile failure with the patch -> going to push existing patch to S3_0+, and will do nothing to S2_6.

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

編集

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