city_create_unit() segfaults when punit is null

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

詳細

This wasn't caught before because it's very rare. However, there may be ways to get units which are null in some situations like multiple build slots, etc. FCW server experienced it and had to fix it.

 static struct unit *city_create_unit(struct city *pcity,
                                      const struct unit_type *utype)
 {
   if (!pcity || !utype) return NULL;   // 17Oct2022 attempt to avoid segfault
 
   struct player *pplayer = city_owner(pcity);
   struct unit *punit;
   int saved_unit_id;
                       city_production_unit_veteran_level(pcity, utype),
                       pcity->id, 0);
   pplayer->score.units_built++;
   if (!punit) {
           notify_conn(game.est_connections, city_tile(pcity),
                   E_WONDER_WILL_BE_BUILT, ftc_server,
                   _("Notice: %s in %s creating illegal segfault. Report to admin immediately!"),
                   utype_name_translation(utype), 
                   city_link(pcity));
     return NULL;              // 17Oct2022 line below was segfault:
   }
   saved_unit_id = punit->id;

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

2022-10-17 20:58 更新者: None

• 新しいチケット "city_create_unit() segfaults when punit is null" が作成されました

2022-10-17 21:35 更新者: cazfi

コメント

Reply To (Anonymous)

This wasn't caught before because it's very rare. However, there may be ways to get units which are null in some situations like multiple build slots, etc. FCW server experienced it and had to fix it.

Those cases with multiple build slots producing NULL units seem like result of FCW specific bugs #45894 and #45895

2023-06-09 07:34 更新者: cazfi

• 添付ファイル 0033-Give-city_create_unit-parameters-nonnull-attribute.patch (File ID: 12609) が付加されました

2023-06-09 07:36 更新者: cazfi

• 担当者 が (未割り当て) から cazfi に更新されました
• 解決法 が なし から 受領 に更新されました
• マイルストーン が (未割り当て) から 3.1.0-beta3 (完了済み) に更新されました
• コンポーネント が (未割り当て) から Server に更新されました
• チケットの種類 が バグ から パッチ に更新されました

コメント

Reply To cazfi

Those cases with multiple build slots producing NULL units seem like result of FCW specific bugs #45894 and #45895

Attached patch gives those parameters nonnull attribute, to make it likely that in the future such bugs get caught as soon as they get created.

2023-06-12 03:23 更新者: cazfi

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