This is an automated email from Gerrit.

Stefan Mahr ([email protected]) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/732

-- gerrit

commit 93aba5128c17f834792bfae21c5b47cd6d04c5a5
Author: Stefan Mahr <[email protected]>
Date:   Thu Jun 28 00:53:54 2012 +0200

    jtag: fix opendous reset command
    
    This cast to uint16_t is not (host) endianess save.
    Depending on compiler usb_out_buffer[1] may be undefined.
    
    Change-Id: If686e5d5da39541329c340bbdef472ee7ab0281c
    Signed-off-by: Stefan Mahr <[email protected]>

diff --git a/src/jtag/drivers/opendous.c b/src/jtag/drivers/opendous.c
index 0bb6da6..1b74bd5 100644
--- a/src/jtag/drivers/opendous.c
+++ b/src/jtag/drivers/opendous.c
@@ -465,7 +465,8 @@ void opendous_simple_command(uint8_t command, uint8_t _data)
 
        DEBUG_JTAG_IO("0x%02x 0x%02x", command, _data);
 
-       usb_out_buffer[0] = (uint16_t) 2;
+       usb_out_buffer[0] = 2;
+       usb_out_buffer[1] = 0;
        usb_out_buffer[2] = command;
        usb_out_buffer[3] = _data;
 

-- 

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to