diff options
| author | Ian Romanick <[email protected]> | 2013-01-23 23:35:19 -0500 |
|---|---|---|
| committer | Ian Romanick <[email protected]> | 2013-01-24 13:57:46 -0500 |
| commit | ac158f8ee78fd5902cfeb8712a72cfa4da7ef6f6 (patch) | |
| tree | 80d56d011f3044301b660536cad2f14386eb5cd9 | |
| parent | 7a23029b2feaa915a075a5832e13f6173b5f3c08 (diff) | |
intel: callocing a 32 byte temp is silly, so don't
I believe that the size used to vary, so the dynamic allocation is
necessary.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Anuj Phogat <[email protected]>
Reviewed-by: Matt Turner <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/intel/intel_context.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/drivers/dri/intel/intel_context.c index ab97d664f36..3aa35e6d7f5 100644 --- a/src/mesa/drivers/dri/intel/intel_context.c +++ b/src/mesa/drivers/dri/intel/intel_context.c @@ -984,8 +984,8 @@ intel_query_dri2_buffers(struct intel_context *intel, __DRIscreen *screen = intel->intelScreen->driScrnPriv; struct gl_framebuffer *fb = drawable->driverPrivate; int i = 0; - const int max_attachments = 4; - unsigned *attachments = calloc(2 * max_attachments, sizeof(unsigned)); + unsigned attachments[8]; + const int max_attachments = ARRAY_SIZE(attachments) / 2; struct intel_renderbuffer *front_rb; struct intel_renderbuffer *back_rb; @@ -993,6 +993,7 @@ intel_query_dri2_buffers(struct intel_context *intel, front_rb = intel_get_renderbuffer(fb, BUFFER_FRONT_LEFT); back_rb = intel_get_renderbuffer(fb, BUFFER_BACK_LEFT); + memset(attachments, 0, sizeof(attachments)); if ((intel->is_front_buffer_rendering || intel->is_front_buffer_reading || !back_rb) && front_rb) { @@ -1013,7 +1014,6 @@ intel_query_dri2_buffers(struct intel_context *intel, attachments, i / 2, buffer_count, drawable->loaderPrivate); - free(attachments); } /** |