[v3,18/23] aarch64: Use l_searchlist.r_list for gcs

Message ID 20241023083920.466015-19-yury.khrustalev@arm.com
State Superseded
Delegated to: Wilco Dijkstra
Headers
Series aarch64: Add support for Guarded Control Stack extension |

Checks

Context Check Description
redhat-pt-bot/TryBot-apply_patch success Patch applied to master at the time it was sent
linaro-tcwg-bot/tcwg_glibc_build--master-aarch64 success Build passed
linaro-tcwg-bot/tcwg_glibc_check--master-aarch64 success Test passed
linaro-tcwg-bot/tcwg_glibc_build--master-arm success Build passed
linaro-tcwg-bot/tcwg_glibc_check--master-arm success Test passed

Commit Message

Yury Khrustalev Oct. 23, 2024, 8:39 a.m. UTC
  From: Szabolcs Nagy <szabolcs.nagy@arm.com>

Allows using the same function for static exe.
---
 sysdeps/aarch64/dl-gcs.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Andreas Schwab Oct. 23, 2024, 9:07 a.m. UTC | #1
Please fold into 17.
  
Yury Khrustalev Nov. 29, 2024, 4:42 p.m. UTC | #2
Hi Andreas

On Wed, Oct 23, 2024 at 11:07:10AM +0200, Andreas Schwab wrote:
> Please fold into 17.

Thank you for your comment. I've addressed this in v4:
https://inbox.sourceware.org/libc-alpha/20241129163721.2385847-1-yury.khrustalev@arm.com/

Kind regards,
Yury
  

Patch

diff --git a/sysdeps/aarch64/dl-gcs.c b/sysdeps/aarch64/dl-gcs.c
index a92deb54b5..764b8a56e9 100644
--- a/sysdeps/aarch64/dl-gcs.c
+++ b/sysdeps/aarch64/dl-gcs.c
@@ -55,5 +55,5 @@  _dl_gcs_check (struct link_map *l, const char *program)
 
   check_gcs (l, program);
   for (unsigned int i = 0; i < l->l_searchlist.r_nlist; i++)
-    check_gcs (l->l_initfini[i], program);
+    check_gcs (l->l_searchlist.r_list[i], program);
 }