target/image: allow for comments in IHEX files
authorFranck Jullien <franck.jullien@gmail.com>
Tue, 24 Dec 2013 09:07:14 +0000 (10:07 +0100)
committerSpencer Oliver <spen@spen-soft.co.uk>
Mon, 30 Dec 2013 15:49:15 +0000 (15:49 +0000)
This is not in the Intel hex file format specification but
some hex files may include comments (i.e. Altera USB-Blaster II
firmware) starting with '#'.

This patch makes image_ihex_buffer_complete_inner to skip
comment lines.

Change-Id: Id1f57d84d75da45e592f1c72b2b8c29193bc14e3
Signed-off-by: Franck Jullien <franck.jullien@gmail.com>
Reviewed-on: http://openocd.zylin.com/1842
Tested-by: jenkins
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
src/target/image.c

index f47ba2274b822a93f4a30113152ee2587ec8788b..8bcba349878f885fd70e0487916a42d32febe25b 100644 (file)
@@ -152,6 +152,9 @@ static int image_ihex_buffer_complete_inner(struct image *image,
                uint8_t cal_checksum = 0;
                size_t bytes_read = 0;
 
+               if (lpszLine[0] == '#')
+                       continue;
+
                if (sscanf(&lpszLine[bytes_read], ":%2" SCNx32 "%4" SCNx32 "%2" SCNx32, &count,
                        &address, &record_type) != 3)
                        return ERROR_IMAGE_FORMAT_ERROR;