Re: [Mesa-dev] [PATCH v2] i965: Check return value of screen->image.loader->getBuffers (v2)

Hi Tomasz On 13 June 2016 at 11:53, Tomasz Figa wrote: > The images struct is an uninitialized local variable on the stack. If the > callback returns 0, the struct might not have been updated and so should > be considered uninitialized. Currently the code ignores the return

Re: [Mesa-dev] [PATCH v2] i965: Check return value of screen->image.loader->getBuffers

Please disregard this one. Forgot to add change log and update the subject with version suffix. Fixed and resent already. Sorry for the noise. On Mon, Jun 13, 2016 at 7:48 PM, Tomasz Figa wrote: > The images struct is an uninitialized local variable on the stack. If the >

[Mesa-dev] [PATCH v2] i965: Check return value of screen->image.loader->getBuffers (v2)

The images struct is an uninitialized local variable on the stack. If the callback returns 0, the struct might not have been updated and so should be considered uninitialized. Currently the code ignores the return value, which (depending on stack contents) might end up in reading a non-zero value

[Mesa-dev] [PATCH v2] i965: Check return value of screen->image.loader->getBuffers

The images struct is an uninitialized local variable on the stack. If the callback returns 0, the struct might not have been updated and so should be considered uninitialized. Currently the code ignores the return value, which (depending on stack contents) might end up in reading a non-zero value