output/alsa: default "device" is an empty string

Fixes nullptr dereference in std::string::operator=(). Also fixes a memory leak by using config_get_block_string() instead of config_dup_block_string().

output/alsa: default "device" is an empty string
7e92820c · Max Kellermann · 5692e20f · 7e92820c
Commit 7e92820c authored Feb 04, 2013 by Max Kellermann
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

AlsaOutputPlugin.cxx src/output/AlsaOutputPlugin.cxx +1 -1

No files found.
--- a/src/output/AlsaOutputPlugin.cxx
+++ b/src/output/AlsaOutputPlugin.cxx
@@ -137,7 +137,7 @@ alsa_device(const AlsaOutput *ad)
 static void
 alsa_configure(AlsaOutput *ad, const struct config_param *param)
 {
-	ad->device = config_dup_block_string(param, "device", NULL);
+	ad->device = config_get_block_string(param, "device", "");

 	ad->use_mmap = config_get_block_bool(param, "use_mmap", false);