ap_diplomat_battle() nonnull target_tile

• Informador: cazfi
• Propietario: cazfi
• Tipo: Bugs
• Estado: Cerrado
• Componente: General
• Hito: 3.0.8 (cerrado)
• Prioridad: 5 - Medium
• Gravedad: 5 - Medium
• Resolución: Fixed
• Fichero: 3

Details

Seen on S3_0; clang analyzer gives warnings about NULL dereference with tgt_tile parameter of ap_diplomat_battle().

Should probably make it nonnull, and see what further issues that reveals.

Ticket History (3/6 Histories)

2023-05-24 20:32 Updated by: cazfi

• New Ticket "ap_diplomat_battle() nonnull target_tile" created

2023-05-27 05:19 Updated by: cazfi

• File 0016-Add-nonnull-attribute-to-ap_diplomat_battle-tgt_tile.patch (File ID: 12484) is attached

2023-05-27 05:20 Updated by: cazfi

• File 0009-Add-nonnull-attribute-to-ap_diplomat_battle-tgt_tile.patch (File ID: 12485) is attached

2023-05-27 05:20 Updated by: cazfi

• File 0005-Fix-ap_diplomat_battle-clang-analyzer-warnings.patch (File ID: 12486) is attached

2023-05-27 05:21 Updated by: cazfi

• Propietario Update from (Ninguno) to cazfi
• Resolución Update from Ninguno to Accepted

Comentario

Reply To cazfi

Should probably make it nonnull, and see what further issues that reveals.

S3_1 - main were not affected, but the "nonnull attribute" part done in attached patches anyway. S3_0 patch has more comprehensive fix.

2023-05-29 13:05 Updated by: cazfi

• Estado Update from Open to Cerrado
• Resolución Update from Accepted to Fixed