mirrors/alsa-lib
1
0
Fork 0
mirror of https://github.com/alsa-project/alsa-lib.git synced 2025-10-29 05:40:25 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

ucm: fix some variable constness issues

Browse source 
I submitted earlier a patch that made the value parameter of
snd_use_case_get() non-const, but as that changed the public API, the
patch couldn't be accepted. This is the same patch, modifying the
internal code so that there are fewer issues with constness, but the
public API is left alone (a comment was added to the function
documentation, though, so that hopefully nobody else will try to fix
the same unfixable problem).

Signed-off-by: Tanu Kaskinen <tanu.kaskinen@linux.intel.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Tanu Kaskinen 2015-02-17 21:15:23 +02:00 • committed by Takashi Iwai
parent a102e66440
commit 567c831e69
2 changed files with 18 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

5
include/use-case.h
View file

@ -224,8 +224,9 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
* \param value Value pointer
* \return Zero if success, otherwise a negative error code
*
* Note: String is dynamically allocated, use free() to
* deallocate this string.
* Note: The returned string is dynamically allocated, use free() to
* deallocate this string. (Yes, the value parameter shouldn't be marked as
* "const", but it's too late to fix it, sorry about that.)
*
* Known identifiers:
* NULL - return current card

29
src/ucm/main.c
View file

@ -40,9 +40,9 @@
* misc
*/
static int get_value1(const char **value, struct list_head *value_list,
static int get_value1(char **value, struct list_head *value_list,
const char *identifier);
static int get_value3(const char **value,
static int get_value3(char **value,
const char *identifier,
struct list_head *value_list1,
struct list_head *value_list2,
@ -299,8 +299,8 @@ static int execute_sequence(snd_use_case_mgr_t *uc_mgr,
case SEQUENCE_ELEMENT_TYPE_CSET:
case SEQUENCE_ELEMENT_TYPE_CSET_BIN_FILE:
if (cdev == NULL) {
const char *playback_ctl = NULL;
const char *capture_ctl = NULL;
char *playback_ctl = NULL;
char *capture_ctl = NULL;
err = get_value3(&playback_ctl, "PlaybackCTL",
value_list1,
@ -315,7 +315,7 @@ static int execute_sequence(snd_use_case_mgr_t *uc_mgr,
value_list2,
value_list3);
if (err < 0 && err != -ENOENT) {
free((char *)playback_ctl);
free(playback_ctl);
uc_error("cdev is not defined!");
return err;
}
@ -327,16 +327,16 @@ static int execute_sequence(snd_use_case_mgr_t *uc_mgr,
if (playback_ctl != NULL &&
capture_ctl != NULL &&
strcmp(playback_ctl, capture_ctl) != 0) {
free((char *)playback_ctl);
free((char *)capture_ctl);
free(playback_ctl);
free(capture_ctl);
uc_error("cdev is not defined!");
return -EINVAL;
}
if (playback_ctl != NULL) {
cdev = (char *)playback_ctl;
free((char *)capture_ctl);
cdev = playback_ctl;
free(capture_ctl);
} else
cdev = (char *)capture_ctl;
cdev = capture_ctl;
}
if (ctl == NULL) {
err = open_ctl(uc_mgr, &ctl, cdev);
@ -1237,7 +1237,7 @@ int snd_use_case_get_list(snd_use_case_mgr_t *uc_mgr,
return err;
}
static int get_value1(const char **value, struct list_head *value_list,
static int get_value1(char **value, struct list_head *value_list,
const char *identifier)
{
struct ucm_value *val;
@ -1258,7 +1258,7 @@ static int get_value1(const char **value, struct list_head *value_list,
return -ENOENT;
}
static int get_value3(const char **value,
static int get_value3(char **value,
const char *identifier,
struct list_head *value_list1,
struct list_head *value_list2,
@ -1288,7 +1288,7 @@ static int get_value3(const char **value,
*/
static int get_value(snd_use_case_mgr_t *uc_mgr,
const char *identifier,
const char **value,
char **value,
const char *mod_dev_name,
const char *verb_name,
int exact)
@ -1419,7 +1419,8 @@ int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
verb = NULL;
}
err = get_value(uc_mgr, ident, value, mod_dev, verb, exact);
err = get_value(uc_mgr, ident, (char **)value, mod_dev, verb,
exact);
if (ident != identifier)
free((void *)ident);
if (mod_dev)