[PATCH for-4.14] kdd: fix build again

Wei Liu posted 1 patch 3 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/xen tags/patchew/20200703201001.56606-1-wl@xen.org
Maintainers: Ian Jackson <ian.jackson@eu.citrix.com>, Wei Liu <wl@xen.org>, Tim Deegan <tim@xen.org>
tools/debugger/kdd/kdd.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH for-4.14] kdd: fix build again
Posted by Wei Liu 3 years, 8 months ago
Restore Tim's patch. The one that was committed was recreated by me
because git didn't accept my saved copy. I made some mistakes while
recreating that patch and here we are.

Fixes: 3471cafbdda3 ("kdd: stop using [0] arrays to access packet contents")
Reported-by: Michael Young <m.a.young@durham.ac.uk>
Signed-off-by: Wei Liu <wl@xen.org>
---
Cc: Tim Deegan <tim@xen.org>
Cc: Paul Durrant <paul@xen.org>
Cc: Michael Young <m.a.young@durham.ac.uk>
---
 tools/debugger/kdd/kdd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/debugger/kdd/kdd.c b/tools/debugger/kdd/kdd.c
index 866532f0c770..a7d0976ea4a8 100644
--- a/tools/debugger/kdd/kdd.c
+++ b/tools/debugger/kdd/kdd.c
@@ -79,11 +79,11 @@ typedef struct {
 /* State of the debugger stub */
 typedef struct {
     union {
-        uint8_t txb[sizeof (kdd_hdr)];           /* Marshalling area for tx */
+        uint8_t txb[sizeof (kdd_pkt)];           /* Marshalling area for tx */
         kdd_pkt txp;                 /* Also readable as a packet structure */
     };
     union {
-        uint8_t rxb[sizeof (kdd_hdr)];           /* Marshalling area for rx */
+        uint8_t rxb[sizeof (kdd_pkt)];           /* Marshalling area for rx */
         kdd_pkt rxp;                 /* Also readable as a packet structure */
     };
     unsigned int cur;       /* Offset into rx where we'll put the next byte */
-- 
2.20.1


Re: [PATCH for-4.14] kdd: fix build again
Posted by Tim Deegan 3 years, 8 months ago
At 20:10 +0000 on 03 Jul (1593807001), Wei Liu wrote:
> Restore Tim's patch. The one that was committed was recreated by me
> because git didn't accept my saved copy. I made some mistakes while
> recreating that patch and here we are.
> 
> Fixes: 3471cafbdda3 ("kdd: stop using [0] arrays to access packet contents")
> Reported-by: Michael Young <m.a.young@durham.ac.uk>
> Signed-off-by: Wei Liu <wl@xen.org>

Reviewed-by: Tim Deegan <tim@xen.org>

Thanks!

Tim.

RE: [PATCH for-4.14] kdd: fix build again
Posted by Paul Durrant 3 years, 8 months ago
> -----Original Message-----
> From: Tim Deegan <tim@xen.org>
> Sent: 03 July 2020 21:27
> To: Wei Liu <wl@xen.org>
> Cc: Xen Development List <xen-devel@lists.xenproject.org>; Michael Young <m.a.young@durham.ac.uk>;
> Paul Durrant <paul@xen.org>; Ian Jackson <ian.jackson@eu.citrix.com>
> Subject: Re: [PATCH for-4.14] kdd: fix build again
> 
> At 20:10 +0000 on 03 Jul (1593807001), Wei Liu wrote:
> > Restore Tim's patch. The one that was committed was recreated by me
> > because git didn't accept my saved copy. I made some mistakes while
> > recreating that patch and here we are.
> >
> > Fixes: 3471cafbdda3 ("kdd: stop using [0] arrays to access packet contents")
> > Reported-by: Michael Young <m.a.young@durham.ac.uk>
> > Signed-off-by: Wei Liu <wl@xen.org>
> 
> Reviewed-by: Tim Deegan <tim@xen.org>
> 
> Thanks!
> 
> Tim.

Release-acked-by: Paul Durrant <paul@xen.org>