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

• Rapporteur: cazfi
• Propriétaire: cazfi
• Type: Bogues
• État: Atteints
• Composant: Server
• Jalon: 3.0.4 (atteints)
• Priorité: 5 - moyen
• Sévérité: 5 - moyen
• Résolution: Fixed
• Fichier: 1

Détails

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

• Propriétaire Update from (Aucun) to cazfi
• Résolution Update from Aucun to Accepted

Commentaire

Going to push also to S2_6

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

• État Update from Ouvert to Atteints
• Résolution Update from Accepted to Fixed