Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
home:zhy20120210:3
alsa
alsa-lib-namehint-fix-corruption.diff
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File alsa-lib-namehint-fix-corruption.diff of Package alsa
From e1c7dd261347f6a0b9ad56e52bb86dfe057cfb9a Mon Sep 17 00:00:00 2001 From: Takashi Iwai <tiwai@suse.de> Date: Tue, 3 Nov 2009 08:57:10 +0100 Subject: [PATCH] Fix corruption after snd_device_name_hint() snd_device_name_hint() corrupts the config name space after its call. This results in the error from the suceeding calls of snd_pcm_open() after snd_device_name_hint(). The bug is in try_config() in namehint.c; it calls snd_config_delete(res) but res can be two different objects in the function. One is the object obtained via snd_config_search_definition(), and another is the one from snd_config_search_alias_hooks(). The former is the expanded objects, thus it should be freed. But, the latter is a reference, and must not be freed. This patch adds the check to free or not. Reported-by: John Lindgren <john.lindgren@tds.net> Signed-off-by: Takashi Iwai <tiwai@suse.de> --- src/control/namehint.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/control/namehint.c b/src/control/namehint.c index e878f83..a134ed7 100644 --- a/src/control/namehint.c +++ b/src/control/namehint.c @@ -219,6 +219,7 @@ static int try_config(struct hint_list *list, const char *str; int err = 0, level; long dev = list->device; + int cleanup_res = 0; list->device_input = -1; list->device_output = -1; @@ -244,6 +245,7 @@ static int try_config(struct hint_list *list, snd_lib_error_set_handler(eh); if (err < 0) goto __skip_add; + cleanup_res = 1; err = -EINVAL; if (snd_config_get_type(res) != SND_CONFIG_TYPE_COMPOUND) goto __cleanup; @@ -330,6 +332,7 @@ static int try_config(struct hint_list *list, goto __hint; snd_config_delete(res); res = NULL; + cleanup_res = 0; if (strchr(buf, ':') != NULL) goto __ok; /* find, if all parameters have a default, */ @@ -379,7 +382,7 @@ static int try_config(struct hint_list *list, err = hint_list_add(list, buf, buf1); } __skip_add: - if (res) + if (res && cleanup_res) snd_config_delete(res); if (buf1) free(buf1); -- 1.6.6.1
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor