Module: Mesa Branch: master Commit: 4201b7d1bf4f3358c944065dd79c07fbdf561dbb URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=4201b7d1bf4f3358c944065dd79c07fbdf561dbb
Author: Lionel Landwerlin <[email protected]> Date: Mon May 8 18:47:06 2017 +0100 intel: gen decoder: don't check for size_t negative values We should get either 0 or 1 here. CID: 1373562 (Control flow issues) Signed-off-by: Lionel Landwerlin <[email protected]> Acked-by: Matt Turner <[email protected]> --- src/intel/common/gen_decoder.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c index 15bba3274e..24147340f5 100644 --- a/src/intel/common/gen_decoder.c +++ b/src/intel/common/gen_decoder.c @@ -648,7 +648,7 @@ gen_spec_load_from_path(const struct gen_device_info *devinfo, do { buf = XML_GetBuffer(ctx.parser, XML_BUFFER_SIZE); len = fread(buf, 1, XML_BUFFER_SIZE, input); - if (len < 0) { + if (len == 0) { fprintf(stderr, "fread: %m\n"); fclose(input); free(filename); _______________________________________________ mesa-commit mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-commit
