From 096150fa19014b397a5d8f8d774bb8236ac37679 Mon Sep 17 00:00:00 2001
From: Shyamanth RH <shyamanth.rh@arm.com>
Date: Wed, 4 Jan 2023 13:08:35 +0530
Subject: [PATCH] Fix for issue #245

* The change fixes the build issue observed in GCC 12.XX.
* Looks like GCC is confusing label to a local variable and hence triggers dangling-pointer error when a label addres is assigned to a pointer.
* Changed branch_to_test from void * pointer to uint64_t datatype since we just need the retrun address of the label while updating the ELR. This should suppress the dangling-pinter warning thrown by GCC 12.XX

Signed-off-by: Shyamanth RH <shyamanth.rh@arm.com>
Upstream-Status: Backport
---
 test_pool/peripherals/test_d003.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/test_pool/peripherals/test_d003.c b/test_pool/peripherals/test_d003.c
index 9f36e1f..0658a45 100755
--- a/test_pool/peripherals/test_d003.c
+++ b/test_pool/peripherals/test_d003.c
@@ -30,7 +30,7 @@
 
 static uint64_t l_uart_base;
 static uint32_t int_id;
-static void *branch_to_test;
+static uint64_t branch_to_test;
 static uint32_t test_fail;
 
 static
@@ -40,7 +40,7 @@ esr(uint64_t interrupt_type, void *context)
   uint32_t index = val_pe_get_index_mpid(val_pe_get_mpid());
 
   /* Update the ELR to point to next instrcution */
-  val_pe_update_elr(context, (uint64_t)branch_to_test);
+  val_pe_update_elr(context, branch_to_test);
 
   val_print(AVS_PRINT_ERR, "\n       Error : Received Sync Exception type %d", interrupt_type);
   val_set_status(index, RESULT_FAIL(g_sbsa_level, TEST_NUM, 01));
@@ -150,7 +150,7 @@ payload(void)
   val_pe_install_esr(EXCEPT_AARCH64_SYNCHRONOUS_EXCEPTIONS, esr);
   val_pe_install_esr(EXCEPT_AARCH64_SERROR, esr);
 
-  branch_to_test = &&exception_taken;
+  branch_to_test = (uint64_t)&&exception_taken;
 
   if (count == 0) {
       val_print(AVS_PRINT_WARN, "\n       No UART defined by Platform      ", 0);