From: Simon Marchi Date: Tue, 26 Nov 2019 17:12:04 +0000 (-0500) Subject: Fix declaration of sparc_xfer_wcookie X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=cadc9cb88871b636a967b98a2ba86c30a759f544;p=binutils-gdb.git Fix declaration of sparc_xfer_wcookie When building sparc-nat.c with -Wmissing-declarations, we get: CXX sparc-nat.o /home/smarchi/src/binutils-gdb/gdb/sparc-nat.c: In function ‘target_xfer_status sparc_xfer_wcookie(target_ops*, target_object, const char*, gdb_byte*, const gdb_byte*, ULONGEST, ULONGEST, ULONGEST*)’: /home/smarchi/src/binutils-gdb/gdb/sparc-nat.c:255:1: error: no previous declaration for ‘target_xfer_status sparc_xfer_wcookie(target_ops*, target_object, const char*, gdb_byte*, const gdb_byte*, ULONGEST, ULONGEST, ULONGEST*)’ [-Werror=missing-declarations] sparc_xfer_wcookie (struct target_ops *ops, enum target_object object, ^~~~~~~~~~~~~~~~~~ Indeed, the declaration is not in sync with the definition, fix that. sparc_xfer_wcookie is used in sparc_target::xfer_partial. sparc_target is only used in the BSD sparc native files. The error above was obtained by running "make sparc-nat.o" on Linux with a cross-compiler for sparc64-linux-gnu. But I presume that if we were to build for real with a BSD/sparc compiler, we would end up with an undefined symbol. gdb/ChangeLog: * sparc-nat.c (sparc_xfer_wcookie): Sync declaration with definition. Change-Id: Id41e706e5516968ff6a49469ddc48eceb29dd3ea --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ad0f981975c..79c09f95e7f 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2019-11-26 Simon Marchi + + * sparc-nat.c (sparc_xfer_wcookie): Sync declaration with + definition. + 2019-11-26 Simon Marchi * remote-sim.c (simulator_command): Make static, remove diff --git a/gdb/sparc-nat.c b/gdb/sparc-nat.c index 0905c57438a..9b769133771 100644 --- a/gdb/sparc-nat.c +++ b/gdb/sparc-nat.c @@ -252,7 +252,7 @@ sparc_store_inferior_registers (struct regcache *regcache, int regnum) TARGET_OBJECT_WCOOKIE. Fetch StackGhost Per-Process XOR cookie. */ enum target_xfer_status -sparc_xfer_wcookie (struct target_ops *ops, enum target_object object, +sparc_xfer_wcookie (enum target_object object, const char *annex, gdb_byte *readbuf, const gdb_byte *writebuf, ULONGEST offset, ULONGEST len, ULONGEST *xfered_len)