nv50/ir: avoid array overrun when checking for supported mods
authorIlia Mirkin <imirkin@alum.mit.edu>
Sat, 6 Sep 2014 15:41:36 +0000 (11:41 -0400)
committerIlia Mirkin <imirkin@alum.mit.edu>
Tue, 9 Sep 2014 00:06:12 +0000 (20:06 -0400)
Reported by Coverity

Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Cc: "10.2 10.3" <mesa-stable@lists.freedesktop.org>
src/gallium/drivers/nouveau/codegen/nv50_ir_target_nv50.cpp
src/gallium/drivers/nouveau/codegen/nv50_ir_target_nvc0.cpp

index b0d617affa79083a78f2904e9f7ba4d4e5728077..ebcb0ac2a6792bcaf900fb5e7cb4a741421aa4c9 100644 (file)
@@ -449,7 +449,7 @@ TargetNV50::isModSupported(const Instruction *insn, int s, Modifier mod) const
          return false;
       }
    }
-   if (s > 3)
+   if (s >= 3)
       return false;
    return (mod & Modifier(opInfo[insn->op].srcMods[s])) == mod;
 }
index 963b6e47ddfc9081f66e8a08eda1c95809149eb6..817ceb862d790e5f05b36dc82b8cac9dd314d0b3 100644 (file)
@@ -423,7 +423,7 @@ TargetNVC0::isModSupported(const Instruction *insn, int s, Modifier mod) const
          return false;
       }
    }
-   if (s > 3)
+   if (s >= 3)
       return false;
    return (mod & Modifier(opInfo[insn->op].srcMods[s])) == mod;
 }