#88 ANSI sounds crash SyncTERM with All ANSI Music enabled

[Craig Hendricks]

Using SyncTERM 1.1 on Windows 10 64-bit.

The door game Yankee Trader plays sounds at various points. They normally show up like the following strings in terminals without sound support:

BT128O5L48P64CP64C(ascii code 14; music notes)
BO4L32P32CP64CP64CP64L16EP64L32CP64L12E(ascii code 14; music notes)

The way to hear sounds in place of these strings in SycTERM is to set the ANSI music option to "All ANSI Music enabled". The other two options do nothing.

The sounds do play when SyncTERM receives these strings in the game, however SyncTERM then immediately crashes and writes the following to the Application log in Event Viewer:

Faulting application name: syncterm.exe, version: 0.0.0.0, time stamp: 0x00000000
Faulting module name: ntdll.dll, version: 10.0.19041.1741, time stamp: 0x221456c9
Exception code: 0xc0000005
Fault offset: 0x0005f583
Faulting process id: 0x2264
Faulting application start time: 0x01d88b25b366eaae
Faulting application path: C:\Program Files (x86)\SyncTERM\syncterm.exe
Faulting module path: C:\WINDOWS\SYSTEM32\ntdll.dll
Report Id: 5685894c-3191-4bc7-a512-371ff1924027
Faulting package full name: (blank)
Faulting package-relative application ID: (blank)

A similar thing happens in Linux Mint, but instead SyncTERM completely locks up and refuses further input rather than crashing.

Interestingly, I tried the same thing from Windows Vista 32-bit, and the ANSI music plays there with no problems whatsoever. So I wonder if it's a problem running SyncTERM on specific platforms.

[Stephen James Hurd]

I think this may be fixed in the latest git/nightly. If you could download the nightly from syncterm.net and test on Windows, it would be very helpful.

[Craig Hendricks]

I downloaded the nightly and enabled "All ANSI Music enabled". Sounds from multiple doors are now playing just fine without crashing SyncTERM.

(however SDL video output seems different, doesn't scale or resize as it did in 1.1. I realize that's outside the scope of this issue, but wanted to mention)

Thanks for looking into it!

[Stephen James Hurd]

Yeah, 1.2 enforces the correct aspect ratio so to get the old scaling you need to pick an LCD output mode (ie: LCD 80x25).

Thanks for confirming the fix!