[2.6 patch] sound/oss/sonicvibes.c: fix an array overflow

From: Adrian Bunk (bunk_at_stusta.de)
Date: 03/27/05

  • Next message: Adrian Bunk: "[2.6 patch] fs/udf/udftime.c: fix off by one error"
    Date:	Sun, 27 Mar 2005 22:38:32 +0200
    To: linux-kernel@vger.kernel.org
    
    

    This patch fixes an array overflow found by the Coverity checker.

    Signed-off-by: Adrian Bunk <bunk@stusta.de>

    --- linux-2.6.12-rc1-mm1-full/sound/oss/sonicvibes.c.old 2005-03-23 01:53:13.000000000 +0100
    +++ linux-2.6.12-rc1-mm1-full/sound/oss/sonicvibes.c 2005-03-23 01:55:11.000000000 +0100
    @@ -1146,13 +1146,13 @@ static int mixer_ioctl(struct sv_state *
                     for (i = 0; i < SOUND_MIXER_NRDEVICES; i++) {
                             if (!(val & (1 << i)))
                                     continue;
                             if (mixtable[i].rec)
                                     break;
                     }
    - if (!mixtable[i].rec)
    + if (i == SOUND_MIXER_NRDEVICES)
                             return 0;
                     spin_lock_irqsave(&s->lock, flags);
                     frobindir(s, SV_CIMIX_ADCINL, 0x1f, mixtable[i].rec << 5);
                     frobindir(s, SV_CIMIX_ADCINR, 0x1f, mixtable[i].rec << 5);
                     spin_unlock_irqrestore(&s->lock, flags);
                     return 0;

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/


  • Next message: Adrian Bunk: "[2.6 patch] fs/udf/udftime.c: fix off by one error"

    Relevant Pages

    • [PATCH][5/3][ARM] PCI quirks update for ARM
      ... > the architecture) array. ... This patch fixes that, allowing you to ... * PCI IDE controllers use non-standard I/O port decoding, ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: 2.6.0-rc1-mm1
      ... Always copy and save the vlan header in bridge-nf ... This patch fixes it. ... /* Bridge Hooks */ ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [2.6 patch] drivers/net/tulip/dmfe.c: fix check after use
      ... Adrian Bunk wrote: ... > This patch fixes a check after use found by the Coverity checker. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • [2.6 patch] sound/oss/gus_wave.c: fix off by one errors
      ... This patch fixes two off by one errors found by the Coverity checker. ... the variables are later used as indexes for arrays with ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: nfs4 client bug
      ... A proprietary module was ... > ive been running this since i got the url and so far i havent hit it ... so i think its safe to say this patch fixes the issues i was seeing ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)