aboutsummaryrefslogtreecommitdiff
path: root/loader/loader.c
diff options
context:
space:
mode:
authorBogDan Vatra <bogdan@kdab.com>2016-01-04 14:05:49 +0200
committerMark Lobodzinski <mark@lunarg.com>2016-01-04 09:55:43 -0700
commita14468a5e786887a45ab9b0bc23684c3434872cb (patch)
tree691602e31ea51a6b1f45d97aa35e18f3eb149652 /loader/loader.c
parent6b04c70ff168e194949f19cd9a13236fa7666495 (diff)
downloadusermoji-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.c7
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];