[Mesa-dev] [PATCH 08/23] i965: Fix memory leak when serializing nir

Timothy Arceri tarceri at itsqueeze.com
Wed Nov 29 01:24:44 UTC 2017


From: Jordan Justen <jordan.l.justen at intel.com>

Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
Reviewed-by: Timothy Arceri <tarceri at itsqueeze.com>
---
 src/mesa/drivers/dri/i965/brw_program.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mesa/drivers/dri/i965/brw_program.c b/src/mesa/drivers/dri/i965/brw_program.c
index c8513c947da..8d321656fef 100644
--- a/src/mesa/drivers/dri/i965/brw_program.c
+++ b/src/mesa/drivers/dri/i965/brw_program.c
@@ -790,20 +790,21 @@ brw_assign_common_binding_table_offsets(const struct gen_device_info *devinfo,
 void
 brw_program_serialize_nir(struct gl_context *ctx, struct gl_program *prog,
                           gl_shader_stage stage)
 {
    struct blob writer;
    blob_init(&writer);
    nir_serialize(&writer, prog->nir);
    prog->driver_cache_blob = ralloc_size(NULL, writer.size);
    memcpy(prog->driver_cache_blob, writer.data, writer.size);
    prog->driver_cache_blob_size = writer.size;
+   blob_finish(&writer);
 }
 
 void
 brw_program_deserialize_nir(struct gl_context *ctx, struct gl_program *prog,
                             gl_shader_stage stage)
 {
    if (!prog->nir) {
       assert(prog->driver_cache_blob && prog->driver_cache_blob_size > 0);
       const struct nir_shader_compiler_options *options =
          ctx->Const.ShaderCompilerOptions[stage].NirOptions;
-- 
2.14.3



More information about the mesa-dev mailing list