From: Lars-Peter Clausen Date: Wed, 18 Jun 2014 11:32:34 +0000 (+0200) Subject: ALSA: control: Handle numid overflow X-Git-Tag: TizenStudio_2.0_p2.3~23 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=039509e1f543118fb38d5d9968058eef15b7ba06;p=sdk%2Femulator%2Femulator-kernel.git ALSA: control: Handle numid overflow Each control gets automatically assigned its numids when the control is created. The allocation is done by incrementing the numid by the amount of allocated numids per allocation. This means that excessive creation and destruction of controls (e.g. via SNDRV_CTL_IOCTL_ELEM_ADD/REMOVE) can cause the id to eventually overflow. Currently when this happens for the control that caused the overflow kctl->id.numid + kctl->count will also over flow causing it to be smaller than kctl->id.numid. Most of the code assumes that this is something that can not happen, so we need to make sure that it won't happen Change-Id: I1b1a978235ecc95a1551e711da8efa9cac003d4a Signed-off-by: Lars-Peter Clausen Acked-by: Jaroslav Kysela Cc: Signed-off-by: Takashi Iwai --- diff --git a/sound/core/control.c b/sound/core/control.c index 0469ba695c4c..98a29b26c5f4 100644 --- a/sound/core/control.c +++ b/sound/core/control.c @@ -289,6 +289,10 @@ static bool snd_ctl_remove_numid_conflict(struct snd_card *card, { struct snd_kcontrol *kctl; + /* Make sure that the ids assigned to the control do not wrap around */ + if (card->last_numid >= UINT_MAX - count) + card->last_numid = 0; + list_for_each_entry(kctl, &card->controls, list) { if (kctl->id.numid < card->last_numid + 1 + count && kctl->id.numid + kctl->count > card->last_numid + 1) {