[alsa-devel] ASoC: Intel: mrfld: add DSP core controls
Dan Carpenter
dan.carpenter at oracle.com
Wed Oct 29 16:14:57 CET 2014
Hello Vinod Koul,
The patch 24c8d14192cc: "ASoC: Intel: mrfld: add DSP core controls"
from Oct 15, 2014, leads to the following static checker warning:
sound/soc/intel/sst-atom-controls.c:1354 sst_map_modules_to_pipe()
warn: variable dereferenced before check 'platform' (see line 1353)
sound/soc/intel/sst-atom-controls.c
1348 static int sst_map_modules_to_pipe(struct snd_soc_platform *platform)
1349 {
1350 struct snd_soc_dapm_widget *w;
1351 int ret = 0;
1352
1353 list_for_each_entry(w, &platform->component.card->widgets, list) {
^^^^^^^^
New dereference.
1354 if (platform && is_sst_dapm_widget(w) && (w->priv)) {
^^^^^^^^
New check.
1355 struct sst_ids *ids = w->priv;
1356
1357 dev_dbg(platform->dev, "widget type=%d name=%s\n",
1358 w->id, w->name);
1359 INIT_LIST_HEAD(&ids->algo_list);
1360 INIT_LIST_HEAD(&ids->gain_list);
1361 ret = sst_fill_widget_module_info(w, platform);
1362
1363 if (ret < 0)
1364 return ret;
1365
1366 /* fill linked widgets */
1367 if (ids->parent_wname != NULL)
1368 sst_fill_linked_widgets(platform, ids);
1369 }
1370 }
1371 return 0;
1372 }
regards,
dan carpenter
More information about the Alsa-devel
mailing list