Re: [Mesa-dev] [PATCH 4/5] gbm: Export a per plane getter for stride

2017-03-08 Thread Jason Ekstrand
On Tue, Mar 7, 2017 at 9:14 PM, Ben Widawsky wrote: > v2: Preserve legacy behavior when plane is 0 (Jason Ekstrand) > EINVAL when input plane is greater than total planes (Jason Ekstrand) > Don't leak the image after fromPlanar (Daniel) > Move bo->image check below plane count preventing bad inde

[Mesa-dev] [PATCH 4/5] gbm: Export a per plane getter for stride

2017-03-07 Thread Ben Widawsky
v2: Preserve legacy behavior when plane is 0 (Jason Ekstrand) EINVAL when input plane is greater than total planes (Jason Ekstrand) Don't leak the image after fromPlanar (Daniel) Move bo->image check below plane count preventing bad index succeeding (Daniel) v3: Fix DRIimage leak (using Jason's re

Re: [Mesa-dev] [PATCH 4/5] gbm: Export a per plane getter for stride

2017-03-07 Thread Ben Widawsky
On 17-03-07 08:33:32, Jason Ekstrand wrote: On Mon, Mar 6, 2017 at 6:37 PM, Ben Widawsky wrote: v2: Preserve legacy behavior when plane is 0 (Jason Ekstrand) EINVAL when input plane is greater than total planes (Jason Ekstrand) Don't leak the image after fromPlanar (Daniel) Move bo->image chec

Re: [Mesa-dev] [PATCH 4/5] gbm: Export a per plane getter for stride

2017-03-07 Thread Jason Ekstrand
On Mon, Mar 6, 2017 at 6:37 PM, Ben Widawsky wrote: > v2: Preserve legacy behavior when plane is 0 (Jason Ekstrand) > EINVAL when input plane is greater than total planes (Jason Ekstrand) > Don't leak the image after fromPlanar (Daniel) > Move bo->image check below plane count preventing bad inde

[Mesa-dev] [PATCH 4/5] gbm: Export a per plane getter for stride

2017-03-06 Thread Ben Widawsky
v2: Preserve legacy behavior when plane is 0 (Jason Ekstrand) EINVAL when input plane is greater than total planes (Jason Ekstrand) Don't leak the image after fromPlanar (Daniel) Move bo->image check below plane count preventing bad index succeeding (Daniel) v3: Fix DRIimage leak (using Jason's re