ALSA: control: fix numid conflict check for new controls
authorClemens Ladisch <clemens@ladisch.de>
Mon, 7 Mar 2011 12:22:50 +0000 (13:22 +0100)
committerTakashi Iwai <tiwai@suse.de>
Tue, 8 Mar 2011 11:59:48 +0000 (12:59 +0100)
The purpose of the snd_ctl_hole_check() function is to find conflicts
between the numerical IDs of the new control and those of any existing
controls.  However, it would fail to detect an existing control whose
count is smaller than the new control's count and whose interval of IDs
is entirely contained in the interval of the new control's IDs.

To fix this, use the correct formula to detect overlapping intervals,
which happens to simplify the condition.

This problem was not encountered so far because ALSA does not yet allow
drivers to allocate specific control IDs.

Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/core/control.c

index 9ce00ed20fba3d9141cb13dd045230412ee86e4c..d1b5ce818a4ff7bb970b1e15e3b7214cb0224bf1 100644 (file)
@@ -285,10 +285,8 @@ static unsigned int snd_ctl_hole_check(struct snd_card *card,
        struct snd_kcontrol *kctl;
 
        list_for_each_entry(kctl, &card->controls, list) {
-               if ((kctl->id.numid <= card->last_numid &&
-                    kctl->id.numid + kctl->count > card->last_numid) ||
-                   (kctl->id.numid <= card->last_numid + count - 1 &&
-                    kctl->id.numid + kctl->count > card->last_numid + count - 1))
+               if (kctl->id.numid < card->last_numid + 1 + count &&
+                   kctl->id.numid + kctl->count > card->last_numid + 1)
                        return card->last_numid = kctl->id.numid + kctl->count - 1;
        }
        return card->last_numid;