GitHub-Mirror/Star64_linux
2
0
Fork 0
mirror of https://github.com/Fishwaldo/Star64_linux.git synced 2025-04-21 13:53:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

ALSA: control: Simplify snd_ctl_elem_list() implementation

Browse source 
This patch simplifies the code of snd_ctl_elem_list() in the following
ways:

- Avoid a vmalloc() temporary buffer but do copy in each iteration;
  the vmalloc buffer was introduced at the time we took the spinlock
  for the ctl element management.

- Use the standard list_for_each_entry() macro

- Merge two loops into one;
  it used to be a loop for skipping until offset becomes zero and
  another loop to copy the data.  They can be folded into a single
  loop easily.

Reviewed-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Tested-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Takashi Iwai 2017-05-22 17:43:04 +02:00
parent a02cb8f8de
commit 53e7bf4525
1 changed files with 25 additions and 43 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

68
sound/core/control.c
View file

@ -747,11 +747,11 @@ static int snd_ctl_card_info(struct snd_card *card, struct snd_ctl_file * ctl,
static int snd_ctl_elem_list(struct snd_card *card, static int snd_ctl_elem_list(struct snd_card *card,
struct snd_ctl_elem_list __user *_list) struct snd_ctl_elem_list __user *_list)
{ {
struct list_head *plist;
struct snd_ctl_elem_list list; struct snd_ctl_elem_list list;
struct snd_kcontrol *kctl; struct snd_kcontrol *kctl;
struct snd_ctl_elem_id *dst, *id; struct snd_ctl_elem_id id;
unsigned int offset, space, jidx; unsigned int offset, space, jidx;
int err = 0;
if (copy_from_user(&list, _list, sizeof(list))) if (copy_from_user(&list, _list, sizeof(list)))
return -EFAULT; return -EFAULT;
@ -760,52 +760,34 @@ static int snd_ctl_elem_list(struct snd_card *card,
/* try limit maximum space */ /* try limit maximum space */
if (space > 16384) if (space > 16384)
return -ENOMEM; return -ENOMEM;
down_read(&card->controls_rwsem);
list.count = card->controls_count;
list.used = 0;
if (space > 0) { if (space > 0) {
/* allocate temporary buffer for atomic operation */ list_for_each_entry(kctl, &card->controls, list) {
dst = vmalloc(space * sizeof(struct snd_ctl_elem_id)); if (offset >= kctl->count) {
if (dst == NULL) offset -= kctl->count;
return -ENOMEM; continue;
down_read(&card->controls_rwsem); }
list.count = card->controls_count; for (jidx = offset; jidx < kctl->count; jidx++) {
plist = card->controls.next; snd_ctl_build_ioff(&id, kctl, jidx);
while (plist != &card->controls) { if (copy_to_user(list.pids + list.used, &id,
if (offset == 0) sizeof(id))) {
break; err = -EFAULT;
kctl = snd_kcontrol(plist); goto out;
if (offset < kctl->count) }
break; list.used++;
offset -= kctl->count; if (!--space)
plist = plist->next; goto out;
}
list.used = 0;
id = dst;
while (space > 0 && plist != &card->controls) {
kctl = snd_kcontrol(plist);
for (jidx = offset; space > 0 && jidx < kctl->count; jidx++) {
snd_ctl_build_ioff(id, kctl, jidx);
id++;
space--;
list.used++;
} }
plist = plist->next;
offset = 0; offset = 0;
} }
up_read(&card->controls_rwsem);
if (list.used > 0 &&
copy_to_user(list.pids, dst,
list.used * sizeof(struct snd_ctl_elem_id))) {
vfree(dst);
return -EFAULT;
}
vfree(dst);
} else {
down_read(&card->controls_rwsem);
list.count = card->controls_count;
up_read(&card->controls_rwsem);
} }
if (copy_to_user(_list, &list, sizeof(list))) out:
return -EFAULT; up_read(&card->controls_rwsem);
return 0; if (!err && copy_to_user(_list, &list, sizeof(list)))
err = -EFAULT;
return err;
} }
static bool validate_element_member_dimension(struct snd_ctl_elem_info *info) static bool validate_element_member_dimension(struct snd_ctl_elem_info *info)