This is an automated email from Gerrit.

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

-- gerrit

commit 9326eb12472b539c9420cac6a21971773a5d1d25
Author: Tomas Vanek <[email protected]>
Date:   Sun Aug 5 16:39:31 2018 +0200

    flash/not/at91sam4: fix clang static analyzer warning
    
    Change-Id: I5e5319d855c868adfa012f68086f7f809ec5a069
    Signed-off-by: Tomas Vanek <[email protected]>

diff --git a/src/flash/nor/at91sam4.c b/src/flash/nor/at91sam4.c
index c5b31e9..6a4b36b 100644
--- a/src/flash/nor/at91sam4.c
+++ b/src/flash/nor/at91sam4.c
@@ -3091,7 +3091,8 @@ showall:
                }
                if ((who >= 0) && (((unsigned)(who)) < 
pChip->details.n_gpnvms)) {
                        r = FLASHD_GetGPNVM(&(pChip->details.bank[0]), who, &v);
-                       command_print(CMD_CTX, "sam4-gpnvm%u: %u", who, v);
+                       if (r == ERROR_OK)
+                               command_print(CMD_CTX, "sam4-gpnvm%u: %u", who, 
v);
                        return r;
                } else {
                        command_print(CMD_CTX, "sam4-gpnvm invalid GPNVM: %u", 
who);

-- 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to