oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

output/raop: check if the "host" option is present

Browse Source 
Better than dereferencing NULL.
This commit is contained in:
Max Kellermann 2011-08-24 01:46:22 +02:00
parent 82f336a78f
commit bcaff4b844
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/output/raop_output_plugin.c
View File

@ -1187,13 +1187,20 @@ raop_output_init(G_GNUC_UNUSED const struct audio_format *audio_format,
G_GNUC_UNUSED const struct config_param *param, G_GNUC_UNUSED const struct config_param *param,
GError **error_r) GError **error_r)
{ {
const char *host = config_get_block_string(param, "host", NULL);
if (host == NULL) {
g_set_error_literal(error_r, raop_output_quark(), 0,
"missing option 'host'");
return NULL;
}
struct raop_data *rd; struct raop_data *rd;
rd = new_raop_data(error_r); rd = new_raop_data(error_r);
if (rd == NULL) if (rd == NULL)
return NULL; return NULL;
rd->addr = config_get_block_string(param, "host", NULL); rd->addr = host;
rd->rtsp_port = config_get_block_unsigned(param, "port", 5000); rd->rtsp_port = config_get_block_unsigned(param, "port", 5000);
rd->volume = config_get_block_unsigned(param, "volume", 75); rd->volume = config_get_block_unsigned(param, "volume", 75);
return rd; return rd;