OSDN Git Service

i965: Fix memmem compiler warnings.
authorEric Anholt <eric@anholt.net>
Tue, 17 Oct 2017 22:41:25 +0000 (15:41 -0700)
committerEric Anholt <eric@anholt.net>
Tue, 24 Oct 2017 17:51:18 +0000 (10:51 -0700)
commite91c3540fc620b39a16d5bce9fd75aa0ddd7ed7e
tree431871a42483136c8007be921384b5be7487f385
parenteed9685dd6619ec7598e8c3fd81117d36010510d
i965: Fix memmem compiler warnings.

gcc is throwing this warning in my meson build:

../src/intel/compiler/brw_eu_validate.c:50:11: warning
argument 1 null where non-null expected [-Wnonnull]
    return memmem(haystack.str, haystack.len,
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                  needle.str, needle.len) != NULL;
                                  ~~~~~~~~~~~~~~~~~~~~~~~

The first check for CONTAINS has a NULL error_msg.str and 0 len.  The
glibc implementation will exit without looking at any haystack bytes if
haystack.len < needle.len, so this was safe, but silence the warning
anyway by guarding against implementation variablility.

Fixes: 122ef3799d56 ("i965: Only insert error message if not already present")
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/intel/compiler/brw_eu_validate.c