1. 25 Jan, 2011 1 commit
    • Andreas Mohr's avatar
      ALSA: azt3328 - fix broken AZF_FMT_XLATE macro · c9ba374d
      Andreas Mohr authored
      Cleanly revert to non-macro implementation of
      snd_azf3328_codec_setfmt(), to fix last-minute functionality breakage
      induced by following checkpatch.pl recommendations without giving them
      their due full share of thought ("revolting computer, ensuing PEBKAC").
      
      I would like to thank Jiri Slaby for his very timely (in -rc1 even)
      and unexpected (uncommon hardware) "recognition of the dangerous situation"
      due to his very commendable static parser use. :)
      Reported-by: default avatarJiri Slaby <jslaby@suse.cz>
      Signed-off-by: default avatarAndreas Mohr <andi@lisas.de>
      Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
      c9ba374d
  2. 22 Jan, 2011 1 commit
    • Jesper Juhl's avatar
      ALSA: Xonar, CS43xx: Don't overrun static array · 233d84c4
      Jesper Juhl authored
      'cs4398_regs' in 'struct xonar_cs43xx' is an array of 'u8' with a size of
      8. So, this code in sound/pci/oxygen/xonar_cs43xx.c::dump_d1_registers()
      
          		for (i = 2; i <= 8; ++i)
      	  			snd_iprintf(buffer, " %02x", data->cs4398_regs[i]);
      
      will overrun the array when 'i == 8'.
      
      I guess that what's needed to fix it is the trivial patch below, but I
      must admit that I have no idea about this code, so I may very well be
      wrong. Additionally, I have no way to actually test this, so all I know is
      that the below compiles. Someone who actually knows this code should take
      a look before anything is comitted - consider the below (not much more
      than) a bug report.
      Signed-off-by: default avatarJesper Juhl <jj@chaosbits.net>
      Acked-by: Clemens Ladisch <clemens@ladisch.de
      Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
      233d84c4
  3. 21 Jan, 2011 2 commits
  4. 20 Jan, 2011 2 commits
  5. 19 Jan, 2011 3 commits
  6. 18 Jan, 2011 31 commits