chip-tool (or anything on Darwin) beeps every time an mdns resolution is made

[bluebin14]

Problem

chip-tool beeps every time an mdns resolution is made. The beeping is caused by printf'ing the BELL character 0x07.

void ResolveContext::OnNewInterface(uint32_t interfaceId, const char * fullname, const char * hostnameWithDomain, uint16_t port,
                                    uint16_t txtLen, const unsigned char * txtRecord)
{
    ChipLogDetail(Discovery, "Mdns : %s hostname:%s fullname:%s interface: %" PRIu32 " port: %u TXT:\"%.*s\"", __func__,
                  hostnameWithDomain, fullname, interfaceId, port, static_cast<int>(txtLen), txtRecord);

Doing printf %s on an mdns text record is a bad idea since it's not only ASCII, it also has binary uint8 length bytes. When they happen to be 0x07 you get a beep, it's called the BELL character, referred to as control-G or ^G in caret notation.

TXT:"\^GSAI=300\^HSII=5000\^FD=3840\^DCM=1
VP=4874+77\^CT=0"

Proposed Solution

Do not printf random binary data with %s format specifier.