OSDN Git Service

Fix driver conf parsing in snd_config_hook_load_for_all_cards()
authorTakashi Iwai <tiwai@suse.de>
Mon, 10 Aug 2009 12:11:57 +0000 (14:11 +0200)
committerTakashi Iwai <tiwai@suse.de>
Mon, 10 Aug 2009 12:14:10 +0000 (14:14 +0200)
commit96da0c842d14b40ce8e37726b259229634b3aa21
tree0b99dcc9fcd79f141babd7aad3d2dc28ad6109bc
parent34d63b449fbfd13064d0a65b5b8caf17e4251dc1
Fix driver conf parsing in snd_config_hook_load_for_all_cards()

Reported by Kevin Goodsell.

Summary: load_for_all_cards fails with existing configuration elements

In snd_config_hook_load_for_all_cards, the first call to
snd_config_search attempts to locate an existing configuration node with
the name of the driver.  Typically none is found, and everything is
good. However, if such a node is located, the next line assumes it is a
leaf node with type 'string' and calls snd_config_get_string to fetch
the string value. If this fails, the entire hook is abandoned.

Because of this, setting something like the following in asoundrc:

cards.<driver name>.foo 0

is sufficient to disable the entire card-specific configuration.

As a concrete example, I have a HDA-Intel sound card. dmix.conf includes
a way to set period_size, period_time, and periods by using
configuration elements of the form cards.<driver name>.pcm.dmix.<var>.
In ~/.asoundrc I add

cards.HDA-Intel.pcm.dmix.period_size 1024

This will cause HDA-Intel.conf to fail to load, and the pcm defined in
default.conf will fail to find the device-specific pcm
cards.HDA-Intel.pcm.default, and fall back on the default pcm using
plughw.  By attempting to configure dmix, I have disabled it.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
src/conf.c