Mesa (master): nv30: Fix "array subscript is below array bounds" compiler warning

Hans de Goede jwrdegoede at kemper.freedesktop.org
Sat Jul 2 10:22:25 UTC 2016


Module: Mesa
Branch: master
Commit: d7dfd4cb511b3190c74f5a03ca9bd8351ad6d062
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=d7dfd4cb511b3190c74f5a03ca9bd8351ad6d062

Author: Hans de Goede <hdegoede at redhat.com>
Date:   Wed Jun 29 14:32:19 2016 +0200

nv30: Fix "array subscript is below array bounds" compiler warning

gcc6 does not like the trick where we point to one entry before the
array start and then start a while with a pre-increment.

Signed-off-by: Hans de Goede <hdegoede at redhat.com>
Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

---

 src/gallium/drivers/nouveau/nv30/nv30_transfer.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/gallium/drivers/nouveau/nv30/nv30_transfer.c b/src/gallium/drivers/nouveau/nv30/nv30_transfer.c
index 9ecbcd1..5299b70 100644
--- a/src/gallium/drivers/nouveau/nv30/nv30_transfer.c
+++ b/src/gallium/drivers/nouveau/nv30/nv30_transfer.c
@@ -663,8 +663,7 @@ nv30_transfer_rect(struct nv30_context *nv30, enum nv30_transfer_filter filter,
       {}
    };
 
-   method = methods - 1;
-   while ((++method)->possible) {
+   for (method = methods; method->possible; method++) {
       if (method->possible(nv30, filter, src, dst)) {
          method->execute(nv30, filter, src, dst);
          return;




More information about the mesa-commit mailing list