diff options
| author | BogDan Vatra <bogdan@kdab.com> | 2016-01-04 14:05:49 +0200 |
|---|---|---|
| committer | Mark Lobodzinski <mark@lunarg.com> | 2016-01-04 09:55:43 -0700 |
| commit | a14468a5e786887a45ab9b0bc23684c3434872cb (patch) | |
| tree | 691602e31ea51a6b1f45d97aa35e18f3eb149652 /loader/loader.c | |
| parent | 6b04c70ff168e194949f19cd9a13236fa7666495 (diff) | |
| download | usermoji-a14468a5e786887a45ab9b0bc23684c3434872cb.tar.xz | |
loader: MR111, Fix memleaks
[instance|dev] activated_layer_list is always allocated so, we need to always free it.
Diffstat (limited to 'loader/loader.c')
| -rw-r--r-- | loader/loader.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/loader/loader.c b/loader/loader.c index eb243986..9ab47294 100644 --- a/loader/loader.c +++ b/loader/loader.c @@ -1131,8 +1131,7 @@ static void loader_destroy_logical_device(const struct loader_instance *inst, struct loader_device *dev) { loader_heap_free(inst, dev->app_extension_props); - if (dev->activated_layer_list.count) - loader_destroy_layer_list(inst, &dev->activated_layer_list); + loader_destroy_layer_list(inst, &dev->activated_layer_list); loader_heap_free(inst, dev); } @@ -2730,10 +2729,6 @@ static void loader_add_layer_env( void loader_deactivate_instance_layers(struct loader_instance *instance) { - if (!instance->activated_layer_list.count) { - return; - } - /* Create instance chain of enabled layers */ for (uint32_t i = 0; i < instance->activated_layer_list.count; i++) { struct loader_layer_properties *layer_prop = &instance->activated_layer_list.list[i]; |
