"section_list_destroy(sec);" with garbage 'sec'

• Relator: cazfi
• Dono: cazfi
• Tipo: Bugs
• Estado: Fechado
• Componente: Server
• Marcos: 3.0.4 (fechado)
• Prioridade: 5 - Medium
• Gravidade: 5 - Medium
• Resolução: Fixed
• Arquivo: 1

Details

From clang analyzer report from S3_0.

load_ruleset_nations() does not set 'sec' if there's issues detected in earlier part of the function (it gets set only within later "if (ok) {" -block). Regardless, in the cleanup phase there's an assumption that if it's not NULL, it's leftover from abort within the block that uses it.

/* Clean up on aborted load */
if (sec) {

fc_assert(!ok);
section_list_destroy(sec);

}

Ticket History (3/4 Histories)

2022-09-01 00:32 Updated by: cazfi

• New Ticket ""section_list_destroy(sec);" with garbage 'sec'" created

2022-09-25 20:22 Updated by: cazfi

• File 0045-load_ruleset_nations-Fix-section_list_destroy-call-w.patch (File ID: 10395) is attached

2022-09-25 20:23 Updated by: cazfi

• Dono Update from (Nenhum) to cazfi
• Resolução Update from Nenhum to Accepted

Comentário

Going to push also to S2_6

2022-10-02 13:25 Updated by: cazfi

• Estado Update from Aberto to Fechado
• Resolução Update from Accepted to Fixed