This is an automated email from Gerrit.

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

-- gerrit

commit 79eab523bfecc8a24e65f3cdc3e3c4f3eed78f5a
Author: Hsiangkai Wang <[email protected]>
Date:   Tue Apr 23 18:42:07 2013 +0800

    gdb server: Fix bug. Parse 'M' packet error.
    
    The format of 'M' packet is 'M addr,length:XX...'. The data
    follows ':' immediately. No need to '+2' to SEPARATOR in
    unhexify(), because SEPARATOR points to data correctly.
    
    Change-Id: I15b5758b540816cc727752e7bf68cd45e623f603
    Signed-off-by: Hsiangkai Wang <[email protected]>

diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c
index cb96bf2..e7f5428 100644
--- a/src/server/gdb_server.c
+++ b/src/server/gdb_server.c
@@ -1259,7 +1259,7 @@ static int gdb_write_memory_packet(struct connection 
*connection,
 
        LOG_DEBUG("addr: 0x%8.8" PRIx32 ", len: 0x%8.8" PRIx32 "", addr, len);
 
-       if (unhexify((char *)buffer, separator + 2, len) != (int)len)
+       if (unhexify((char *)buffer, separator, len) != (int)len)
                LOG_ERROR("unable to decode memory packet");
 
        retval = target_write_buffer(target, addr, len, buffer);

-- 

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to