win32 crash on unit_activity_text()

• Reporter: cazfi
• 소유자: cazfi
• Type: Bugs
• Status: Closed
• Component: General
• MileStone: 3.0.6 (closed)
• Priority: 5 - Medium
• Severity: 5 - Medium
• Resolution: Fixed
• File: 4

Details

win32 gtk3.22-client crashes when entering game. Seen on S3_1 and master (where we don't support win32 as such, but underlying bug might be just dormant on other environments).

Debugger reveals that the crash happens on unit_activity_text() call of astr_clear().

unit_activity_text() header has "FIXME: Convert all callers of this function to unit_activity_astr() because this function is not re-entrant." - it's quite possible that this lack of being re-entrant is the cause of this crash.

Ticket History (3/7 Histories)

2023-01-21 11:34 Updated by: cazfi

• New Ticket "win32 crash on unit_activity_text()" created

2023-01-21 12:44 Updated by: cazfi

• File 0033-Retire-unit_activity_text.patch (File ID: 11423) is attached

2023-01-21 12:45 Updated by: cazfi

• File 0016-Retire-unit_activity_text.patch (File ID: 11424) is attached

2023-01-21 12:45 Updated by: cazfi

• File 0012-Retire-unit_activity_text.patch (File ID: 11425) is attached

2023-01-21 12:45 Updated by: cazfi

• File 0002-Retire-unit_activity_text.patch (File ID: 11426) is attached

2023-01-21 12:46 Updated by: cazfi

• 소유자 Update from (None) to cazfi
• Resolution Update from None to Accepted
• Milestone Update from (None) to 3.0.6 (closed)

댓글 올리기

Reply To cazfi

it's quite possible that this lack of being re-entrant is the cause of this crash.

Yes, getting rid of the unit_activity_text() was the solution. Patches attached for all branches, including S2_6.

2023-01-24 08:43 Updated by: cazfi

• Status Update from Open to Closed
• Resolution Update from Accepted to Fixed