[PATCH 2/2] cpukit/libdl: Resolve size mismatch warnings

Kinsey Moore kinsey.moore at oarcorp.com
Tue Apr 4 21:33:59 UTC 2023


Resolve warnings about mismatched pointer and integer sizes in AArch64
libdl when building with the ILP32 ABI.
---
 cpukit/libdl/rtl-elf.c             |  4 ++--
 cpukit/libdl/rtl-mdreloc-aarch64.c | 31 +++++++++++++++---------------
 cpukit/libdl/rtl-rap.c             |  4 ++--
 3 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/cpukit/libdl/rtl-elf.c b/cpukit/libdl/rtl-elf.c
index 8a21c5df73..5754070518 100644
--- a/cpukit/libdl/rtl-elf.c
+++ b/cpukit/libdl/rtl-elf.c
@@ -192,7 +192,7 @@ rtems_rtl_elf_find_symbol (rtems_rtl_obj*      obj,
     if (!*symbol)
       return false;
 
-    *value = (Elf_Addr) (*symbol)->value;
+    *value = (Elf_Addr)(uintptr_t) (*symbol)->value;
     return true;
   }
 
@@ -202,7 +202,7 @@ rtems_rtl_elf_find_symbol (rtems_rtl_obj*      obj,
   if (!sect)
     return false;
 
-  *value = sym->st_value + (Elf_Addr) sect->base;
+  *value = sym->st_value + (Elf_Addr)(uintptr_t) sect->base;
 
   return true;
 }
diff --git a/cpukit/libdl/rtl-mdreloc-aarch64.c b/cpukit/libdl/rtl-mdreloc-aarch64.c
index e44238e636..25057ce9d7 100644
--- a/cpukit/libdl/rtl-mdreloc-aarch64.c
+++ b/cpukit/libdl/rtl-mdreloc-aarch64.c
@@ -107,7 +107,7 @@ checkoverflow(Elf_Addr addr, int bitwidth, Elf_Addr targetaddr,
   const Elf_Addr mask = ~__BITS(bitwidth - 1, 0);
 
   if (((addr & mask) != 0) && ((addr & mask) != mask)) {
-    printf("kobj_reloc: Relocation 0x%jx too far from %p"
+    printf("kobj_reloc: Relocation 0x%" PRIxPTR " too far from %p"
         " (base+0x%jx) for %dbit%s\n",
         (uintptr_t)targetaddr, where, off, bitwidth, bitscale);
     return true;
@@ -120,7 +120,7 @@ static inline bool
 checkalign(Elf_Addr addr, int alignbyte, void *where, Elf64_Addr off)
 {
   if ((addr & (alignbyte - 1)) != 0) {
-    printf("kobj_reloc: Relocation 0x%jx unaligned at %p"
+    printf("kobj_reloc: Relocation 0x%" PRIxPTR " unaligned at %p"
         " (base+0x%jx). must be aligned %d\n",
         (uintptr_t)addr, where, off, alignbyte);
     return true;
@@ -257,7 +257,7 @@ rtems_rtl_elf_reloc_rela (rtems_rtl_obj*            obj,
 
         if (rtems_rtl_trace (RTEMS_RTL_TRACE_RELOC))
           printf ("rtl: reloc 64/GLOB_DAT in %s --> %p in %s\n",
-                  sect->name, (void *)*where,
+                  sect->name, (void *)(uintptr_t)*where,
                   rtems_rtl_obj_oname (obj));
       }
       break;
@@ -270,10 +270,10 @@ rtems_rtl_elf_reloc_rela (rtems_rtl_obj*            obj,
      */
     case R_TYPE(RELATIVE):  /* Delta(S) + A */
       if (!parsing) {
-        *where = (Elf_Addr)(sect->base + rela->r_addend);
+        *where = (Elf_Addr)(uintptr_t)(sect->base + rela->r_addend);
         if (rtems_rtl_trace (RTEMS_RTL_TRACE_RELOC))
           printf ("rtl: reloc RELATIVE in %s --> %p in %s\n",
-                  sect->name, (void *)*where,
+                  sect->name, (void *)(uintptr_t)*where,
                   rtems_rtl_obj_oname (obj));
       }
       break;
@@ -304,7 +304,7 @@ rtems_rtl_elf_reloc_rela (rtems_rtl_obj*            obj,
           shift = 12;
           break;
         default:
-          printf("illegal rtype: %ld\n", ELF_R_TYPE(rela->r_info));
+          printf("illegal rtype: %" PRIu64 "\n", ELF_R_TYPE(rela->r_info));
           break;
       }
 
@@ -344,7 +344,7 @@ rtems_rtl_elf_reloc_rela (rtems_rtl_obj*            obj,
           shift = 3;
           break;
         default:
-          printf("illegal rtype: %ld\n", ELF_R_TYPE(rela->r_info));
+          printf("illegal rtype: %" PRIu64 "\n", ELF_R_TYPE(rela->r_info));
           break;
       }
 
@@ -360,9 +360,9 @@ rtems_rtl_elf_reloc_rela (rtems_rtl_obj*            obj,
         target = (Elf_Addr)symvalue + rela->r_addend;
         if (checkalign(target, 1 << shift, where, off)) {
           printf ("rtl: reloc checkalign failed in %s --> %p in %s\n",
-                  sect->name, (void *)*where,
+                  sect->name, (void *)(uintptr_t)*where,
                   rtems_rtl_obj_oname (obj));
-          printf("ELF_R_TYPE is : %ld\n", ELF_R_TYPE(rela->r_info));
+          printf("ELF_R_TYPE is : %" PRIu64 "\n", ELF_R_TYPE(rela->r_info));
           break;
         }
         target &= WIDTHMASK(12);
@@ -433,7 +433,7 @@ rtems_rtl_elf_reloc_rela (rtems_rtl_obj*            obj,
           return rtems_rtl_elf_rel_failure;
         }
 
-        tramp_addr = ((Elf_Addr) obj->tramp_brk) | (symvalue & 1);
+        tramp_addr = ((Elf_Addr)(uintptr_t)obj->tramp_brk) | (symvalue & 1);
         obj->tramp_brk = set_veneer(obj->tramp_brk, symvalue);
 
         target = tramp_addr + rela->r_addend - (uintptr_t)where;
@@ -468,29 +468,30 @@ rtems_rtl_elf_reloc_rela (rtems_rtl_obj*            obj,
 
     case R_TYPE(TLSDESC):
       printf ("rtl: reloc TLSDESC in %s --> %p in %s\n",
-              sect->name, (void *)*where,
+              sect->name, (void *)(uintptr_t)*where,
               rtems_rtl_obj_oname (obj));
       break;
 
     case R_TLS_TYPE(TLS_DTPREL):
       printf ("rtl: reloc TLS_DTPREL in %s --> %p in %s\n",
-              sect->name, (void *)*where,
+              sect->name, (void *)(uintptr_t)*where,
               rtems_rtl_obj_oname (obj));
       break;
     case R_TLS_TYPE(TLS_DTPMOD):
       printf ("rtl: reloc TLS_DTPMOD in %s --> %p in %s\n",
-              sect->name, (void *)*where,
+              sect->name, (void *)(uintptr_t)*where,
               rtems_rtl_obj_oname (obj));
       break;
 
     case R_TLS_TYPE(TLS_TPREL):
       printf ("rtl: reloc TLS_TPREL in %s --> %p in %s\n",
-              sect->name, (void *)*where,
+              sect->name, (void *)(uintptr_t)*where,
               rtems_rtl_obj_oname (obj));
       break;
 
     default:
-      printf ("rtl: Unsupported relocation type (%ld) in %s --> %p in %s\n",
+      printf ("rtl: Unsupported relocation type (%" PRIu64
+              ") in %s --> %p in %s\n",
               ELF_R_TYPE(rela->r_info), sect->name, (void *)where,
               rtems_rtl_obj_oname (obj));
       return rtems_rtl_elf_rel_failure;
diff --git a/cpukit/libdl/rtl-rap.c b/cpukit/libdl/rtl-rap.c
index f8e2b18f43..3fd1428bf2 100644
--- a/cpukit/libdl/rtl-rap.c
+++ b/cpukit/libdl/rtl-rap.c
@@ -350,7 +350,7 @@ rtems_rtl_rap_relocate (rtems_rtl_rap* rap, rtems_rtl_obj* obj)
           return false;
         }
 
-        symvalue = (Elf_Addr) symsect->base + addend;
+        symvalue = (Elf_Addr)(uintptr_t) symsect->base + addend;
       }
       else if (rtems_rtl_elf_rel_resolve_sym (type))
       {
@@ -390,7 +390,7 @@ rtems_rtl_rap_relocate (rtems_rtl_rap* rap, rtems_rtl_obj* obj)
           return false;
         }
 
-        symvalue = (Elf_Addr) symbol->value;
+        symvalue = (Elf_Addr)(uintptr_t) symbol->value;
       }
 
       if (is_rela)
-- 
2.30.2



More information about the devel mailing list