]> pilppa.com Git - linux-2.6-omap-h63xx.git/commitdiff
ALSA: hda: fix oopses in snd-hda-intel after digital slave support additions
authorHerton Ronaldo Krzesinski <herton@mandriva.com.br>
Sat, 13 Sep 2008 14:44:29 +0000 (16:44 +0200)
committerJaroslav Kysela <perex@perex.cz>
Tue, 23 Sep 2008 06:17:54 +0000 (08:17 +0200)
Many places fail to check if codec has slave_dig_outs entries (the most common
case is not having any entry), leading to various possible oopses in hda_codec
code.

Signed-off-by: Herton Ronaldo Krzesinski <herton@mandriva.com.br>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
sound/pci/hda/hda_codec.c

index 696d77e575ec7765e963a10be20dc479347ec688..531364d353536d015298a9828f0f660e7d16aea2 100644 (file)
@@ -1462,14 +1462,15 @@ static int snd_hda_spdif_default_put(struct snd_kcontrol *kcontrol,
                                          AC_VERB_SET_DIGI_CONVERT_2,
                                          val >> 8);
 
-               for (d = codec->slave_dig_outs; *d; d++) {
-                       snd_hda_codec_write_cache(codec, *d, 0,
+               if (codec->slave_dig_outs)
+                       for (d = codec->slave_dig_outs; *d; d++) {
+                               snd_hda_codec_write_cache(codec, *d, 0,
                                          AC_VERB_SET_DIGI_CONVERT_1,
                                          val & 0xff);
-                       snd_hda_codec_write_cache(codec, *d, 0,
+                               snd_hda_codec_write_cache(codec, *d, 0,
                                          AC_VERB_SET_DIGI_CONVERT_2,
                                          val >> 8);
-               }
+                       }
        }
 
        mutex_unlock(&codec->spdif_mutex);
@@ -1507,8 +1508,9 @@ static int snd_hda_spdif_out_switch_put(struct snd_kcontrol *kcontrol,
                                          AC_VERB_SET_DIGI_CONVERT_1,
                                          val & 0xff);
 
-               for (d = codec->slave_dig_outs; *d; d++)
-                       snd_hda_codec_write_cache(codec, *d, 0,
+               if (codec->slave_dig_outs)
+                       for (d = codec->slave_dig_outs; *d; d++)
+                               snd_hda_codec_write_cache(codec, *d, 0,
                                          AC_VERB_SET_DIGI_CONVERT_1,
                                          val & 0xff);
                /* unmute amp switch (if any) */
@@ -1664,8 +1666,9 @@ static int snd_hda_spdif_in_switch_put(struct snd_kcontrol *kcontrol,
                snd_hda_codec_write_cache(codec, nid, 0,
                                          AC_VERB_SET_DIGI_CONVERT_1, val);
 
-               for (d = codec->slave_dig_outs; *d; d++)
-                       snd_hda_codec_write_cache(codec, *d, 0,
+               if (codec->slave_dig_outs)
+                       for (d = codec->slave_dig_outs; *d; d++)
+                               snd_hda_codec_write_cache(codec, *d, 0,
                                          AC_VERB_SET_DIGI_CONVERT_1, val);
        }
        mutex_unlock(&codec->spdif_mutex);
@@ -2617,9 +2620,10 @@ static void setup_dig_out_stream(struct hda_codec *codec, hda_nid_t nid,
                snd_hda_codec_write(codec, nid, 0, AC_VERB_SET_DIGI_CONVERT_1,
                            codec->spdif_ctls & ~AC_DIG1_ENABLE & 0xff);
 
-               for (d = codec->slave_dig_outs; *d; d++)
-                       snd_hda_codec_write(codec, *d, 0,
-                                       AC_VERB_SET_DIGI_CONVERT_1,
+               if (codec->slave_dig_outs)
+                       for (d = codec->slave_dig_outs; *d; d++)
+                               snd_hda_codec_write(codec, *d, 0,
+                                   AC_VERB_SET_DIGI_CONVERT_1,
                                    codec->spdif_ctls & ~AC_DIG1_ENABLE & 0xff);
        }
        snd_hda_codec_setup_stream(codec, nid, stream_tag, 0, format);
@@ -2628,9 +2632,10 @@ static void setup_dig_out_stream(struct hda_codec *codec, hda_nid_t nid,
                snd_hda_codec_write(codec, nid, 0, AC_VERB_SET_DIGI_CONVERT_1,
                                    codec->spdif_ctls & 0xff);
 
-               for (d = codec->slave_dig_outs; *d; d++)
-                       snd_hda_codec_write(codec, *d, 0,
-                                       AC_VERB_SET_DIGI_CONVERT_1,
+               if (codec->slave_dig_outs)
+                       for (d = codec->slave_dig_outs; *d; d++)
+                               snd_hda_codec_write(codec, *d, 0,
+                                   AC_VERB_SET_DIGI_CONVERT_1,
                                    codec->spdif_ctls & 0xff);
        }