This is an automated email from Gerrit.

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

-- gerrit

commit e97bcf413f8517b66ce907740a9a2b90bac015ba
Author: Spencer Oliver <[email protected]>
Date:   Tue Apr 22 20:53:50 2014 +0100

    nrf51: remove dereference of null pointer
    
    found by clang 3.4.
    
    Change-Id: Id499b546f65acd7a719498bc97e33b21d1ba565a
    Signed-off-by: Spencer Oliver <[email protected]>

diff --git a/src/flash/nor/nrf51.c b/src/flash/nor/nrf51.c
index 2a4b887..adb4004 100644
--- a/src/flash/nor/nrf51.c
+++ b/src/flash/nor/nrf51.c
@@ -629,7 +629,7 @@ static int nrf51_write_page(struct flash_bank *bank, 
uint32_t offset, const uint
        struct flash_sector *sector = nrf51_find_sector_by_address(bank, 
offset);
 
        if (!sector)
-               goto error;
+               return ERROR_FLASH_SECTOR_INVALID;
 
        if (sector->is_protected)
                goto error;

-- 

------------------------------------------------------------------------------
Start Your Social Network Today - Download eXo Platform
Build your Enterprise Intranet with eXo Platform Software
Java Based Open Source Intranet - Social, Extensible, Cloud Ready
Get Started Now And Turn Your Intranet Into A Collaboration Platform
http://p.sf.net/sfu/ExoPlatform
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to