nils/shairport-sync
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Avoid passing NULL pointer to printf(3) %s

Browse Source 
OpenBSD issues syslog(3) warnings about these as they may hint at bugs:
	shairport-sync: vfprintf %s NULL in "metadata socket address is "%s" port %d."

Metadata listener socket is empty by default so use the existing `strnull`.
This commit is contained in:
Klemens Nanni
2024-02-24 14:12:25 +01:00
parent 00a4758404
commit 22fc3bccab
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
shairport.c
+1 -1
View File
@@ -2491,7 +2491,7 @@ int main(int argc, char **argv) {
#ifdef CONFIG_METADATA
debug(1, "metadata enabled is %d.", config.metadata_enabled);
debug(1, "metadata pipename is \"%s\".", config.metadata_pipename);
debug(1, "metadata socket address is \"%s\" port %d.", config.metadata_sockaddr,
debug(1, "metadata socket address is \"%s\" port %d.", strnull(config.metadata_sockaddr),
config.metadata_sockport);
debug(1, "metadata socket packet size is \"%d\".", config.metadata_sockmsglength);
debug(1, "get-coverart is %d.", config.get_coverart);