Re: [Mesa-dev] [PATCH mesa] anv: fix off by one in array check

2017-09-04 Thread Eric Engestrom
On Monday, 2017-09-04 04:54:36 +, Jason Ekstrand wrote: > I sent the same patch a few hours later. I don't care which one we land. > You have a more descriptive commit message. Alright then, I just pushed mine. > > Reviewed-by: Jason Ekstrand > > On Sun, Sep 3, 2017 at 11:33 AM, Eric Enge

Re: [Mesa-dev] [PATCH mesa] anv: fix off by one in array check

2017-09-03 Thread Samuel Iglesias Gonsálvez
Reviewed-by: Samuel Iglesias Gonsálvez On Sun, 2017-09-03 at 21:54 -0700, Jason Ekstrand wrote: > I sent the same patch a few hours later.  I don't care which one we > land.  You have a more descriptive commit message. > > Reviewed-by: Jason Ekstrand > > On Sun, Sep 3, 2017 at 11:33 AM, Eric E

Re: [Mesa-dev] [PATCH mesa] anv: fix off by one in array check

2017-09-03 Thread Jason Ekstrand
I sent the same patch a few hours later. I don't care which one we land. You have a more descriptive commit message. Reviewed-by: Jason Ekstrand On Sun, Sep 3, 2017 at 11:33 AM, Eric Engestrom wrote: > `anv_formats[ARRAY_SIZE(anv_formats)]` is already one too far. > Spotted by Coverity. > > C

[Mesa-dev] [PATCH mesa] anv: fix off by one in array check

2017-09-03 Thread Eric Engestrom
`anv_formats[ARRAY_SIZE(anv_formats)]` is already one too far. Spotted by Coverity. CovID: 1417259 Fixes: 242211933a0682696170 "anv/formats: Nicely handle unknown VkFormat enums" Cc: Jason Ekstrand Signed-off-by: Eric Engestrom --- src/intel/vulkan/anv_formats.c | 2 +- 1 file changed, 1 insert