This is an automated email from Gerrit.

"Mark O'Donovan <[email protected]>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/9498

-- gerrit

commit 4a78a5b4a8455b2b14a747f5715e24d9d67e1fde
Author: Mark O'Donovan <[email protected]>
Date:   Tue Feb 24 22:14:04 2026 +0000

    jtag/drivers/presto: fix potential divide by 0
    
    Found by cppcheck.
    
    Change-Id: I656685eb3b639dc054c3a563203264e03d72d1c7
    Signed-off-by: Mark O'Donovan <[email protected]>

diff --git a/src/jtag/drivers/presto.c b/src/jtag/drivers/presto.c
index 84b881cdc8..552c74c9c9 100644
--- a/src/jtag/drivers/presto.c
+++ b/src/jtag/drivers/presto.c
@@ -451,7 +451,7 @@ static struct bitq_interface presto_bitq = {
 
 static int presto_adapter_khz(int khz, int *jtag_speed)
 {
-       if (khz < 0) {
+       if (khz <= 0) {
                *jtag_speed = 0;
                return ERROR_COMMAND_SYNTAX_ERROR;
        }

-- 

Reply via email to