[PATCH 05/19] drm/amd/display: fix double memory allocation

[Qingqing Zhuo]

From: Martin Leung <martin.le...@amd.com>

[Why & How]
when trying to fix a nullptr dereference on VMs,
accidentally doubly allocated memory for the non VM
case. removed the extra link_srv creation since
dc_construct_ctx is called in both VM and non VM cases
Also added a proper fail check for if kzalloc fails

Cc: sta...@vger.kernel.org
Cc: Mario Limonciello <mario.limoncie...@amd.com>
Reviewed-by: Leo Ma <hanghong...@amd.com>
Acked-by: Qingqing Zhuo <qingqing.z...@amd.com>
Signed-off-by: Martin Leung <martin.le...@amd.com>
---
 drivers/gpu/drm/amd/display/dc/core/dc.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/core/dc.c 
b/drivers/gpu/drm/amd/display/dc/core/dc.c
index 40f2e174c524..52564b93f7eb 100644
--- a/drivers/gpu/drm/amd/display/dc/core/dc.c
+++ b/drivers/gpu/drm/amd/display/dc/core/dc.c
@@ -887,7 +887,10 @@ static bool dc_construct_ctx(struct dc *dc,
        }
 
        dc->ctx = dc_ctx;
+
        dc->link_srv = link_create_link_service();
+       if (!dc->link_srv)
+               return false;
 
        return true;
 }
@@ -986,8 +989,6 @@ static bool dc_construct(struct dc *dc,
                goto fail;
        }
 
-       dc->link_srv = link_create_link_service();
-
        dc->res_pool = dc_create_resource_pool(dc, init_params, 
dc_ctx->dce_version);
        if (!dc->res_pool)
                goto fail;
-- 
2.34.1