Skip to content

Commit

Permalink
gdb_hostio: Convert format-strings to honor argument types (uint32_t,…
Browse files Browse the repository at this point in the history
… size_t, long)
  • Loading branch information
ALTracer committed Nov 15, 2023
1 parent 3d0fe6b commit 678228a
Showing 1 changed file with 11 additions and 10 deletions.
21 changes: 11 additions & 10 deletions src/gdb_hostio.c
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ static int hostio_get_response(target_controller_s *const tc)
/* Interface to host system calls */
int hostio_open(target_controller_s *tc, target_addr_t path, size_t path_len, target_open_flags_e flags, mode_t mode)
{
gdb_putpacket_f("Fopen,%08X/%X,%08X,%08X", path, path_len, flags, mode);
gdb_putpacket_f("Fopen,%08" PRIX32 "/%08" PRIX32 ",%08X,%08" PRIX32, path, (uint32_t)path_len, flags, mode);
return hostio_get_response(tc);
}

Expand All @@ -94,49 +94,50 @@ int hostio_close(target_controller_s *tc, int fd)

int hostio_read(target_controller_s *tc, int fd, target_addr_t buf, unsigned int count)
{
gdb_putpacket_f("Fread,%08X,%08X,%08X", fd, buf, count);
gdb_putpacket_f("Fread,%08X,%08" PRIX32 ",%08X", fd, buf, count);
return hostio_get_response(tc);
}

int hostio_write(target_controller_s *tc, int fd, target_addr_t buf, unsigned int count)
{
gdb_putpacket_f("Fwrite,%08X,%08X,%08X", fd, buf, count);
gdb_putpacket_f("Fwrite,%08X,%08" PRIX32 ",%08X", fd, buf, count);
return hostio_get_response(tc);
}

long hostio_lseek(target_controller_s *tc, int fd, long offset, target_seek_flag_e flag)
{
gdb_putpacket_f("Flseek,%08X,%08X,%08X", fd, offset, flag);
gdb_putpacket_f("Flseek,%08X,%08lX,%08X", fd, offset, flag);
return hostio_get_response(tc);
}

int hostio_rename(target_controller_s *tc, target_addr_t oldpath, size_t old_len, target_addr_t newpath, size_t new_len)
{
gdb_putpacket_f("Frename,%08X/%X,%08X/%X", oldpath, old_len, newpath, new_len);
gdb_putpacket_f(
"Frename,%08" PRIX32 "/%08" PRIX32 ",%08" PRIX32 "/%zX", oldpath, (uint32_t)old_len, newpath, new_len);
return hostio_get_response(tc);
}

int hostio_unlink(target_controller_s *tc, target_addr_t path, size_t path_len)
{
gdb_putpacket_f("Funlink,%08X/%X", path, path_len);
gdb_putpacket_f("Funlink,%08" PRIX32 "/%08" PRIX32, path, (uint32_t)path_len);
return hostio_get_response(tc);
}

int hostio_stat(target_controller_s *tc, target_addr_t path, size_t path_len, target_addr_t buf)
{
gdb_putpacket_f("Fstat,%08X/%X,%08X", path, path_len, buf);
gdb_putpacket_f("Fstat,%08" PRIX32 "/%08" PRIX32 ",%08" PRIX32, path, (uint32_t)path_len, buf);
return hostio_get_response(tc);
}

int hostio_fstat(target_controller_s *tc, int fd, target_addr_t buf)
{
gdb_putpacket_f("Ffstat,%X,%08X", fd, buf);
gdb_putpacket_f("Ffstat,%X,%08" PRIX32, fd, buf);
return hostio_get_response(tc);
}

int hostio_gettimeofday(target_controller_s *tc, target_addr_t tv, target_addr_t tz)
{
gdb_putpacket_f("Fgettimeofday,%08X,%08X", tv, tz);
gdb_putpacket_f("Fgettimeofday,%08" PRIX32 ",%08" PRIX32, tv, tz);
return hostio_get_response(tc);
}

Expand All @@ -148,7 +149,7 @@ int hostio_isatty(target_controller_s *tc, int fd)

int hostio_system(target_controller_s *tc, target_addr_t cmd, size_t cmd_len)
{
gdb_putpacket_f("Fsystem,%08X/%X", cmd, cmd_len);
gdb_putpacket_f("Fsystem,%08" PRIX32 "/%08" PRIX32, cmd, (uint32_t)cmd_len);
return hostio_get_response(tc);
}
#endif /* PC_HOSTED */

0 comments on commit 678228a

Please sign in to comment.