From c5fefb8f892e4a128dcf8b2b96de4fa7cc929d88 Mon Sep 17 00:00:00 2001
From: MCUdude <hansibull@gmail.com>
Date: Wed, 15 Apr 2026 19:27:44 +0200
Subject: [PATCH] Only write to my.vtarg_data if the data is actually valid

---
 src/jtag3.c    | 2 +-
 src/stk500v2.c | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/jtag3.c b/src/jtag3.c
index 2a0f9995..a613ce21 100644
--- a/src/jtag3.c
+++ b/src/jtag3.c
@@ -1566,12 +1566,12 @@ static int jtag3_parseextparms(const PROGRAMMER *pgm, const LISTID extparms) {
           double vtarg_set_val = 0;
           int sscanf_success = sscanf(extended_param, "vtarg=%lf", &vtarg_set_val);
 
-          my.vtarg_data = (double) ((int) (vtarg_set_val*100 + .5))/100;
           if(sscanf_success < 1 || vtarg_set_val < 0) {
             pmsg_error("invalid value in -x %s\n", extended_param);
             rv = -1;
             break;
           }
+          my.vtarg_data = (double) ((int) (vtarg_set_val*100 + .5))/100;
           my.vtarg_set = true;
           continue;
         }
diff --git a/src/stk500v2.c b/src/stk500v2.c
index 2f9f4f95..517575ca 100644
--- a/src/stk500v2.c
+++ b/src/stk500v2.c
@@ -1740,12 +1740,12 @@ static int stk500v2_parseextparms(const PROGRAMMER *pgm, const LISTID extparms)
           double vtarg_set_val = -1;    // Default = invalid value
           int sscanf_success = sscanf(extended_param, "vtarg=%lf", &vtarg_set_val);
 
-          my.vtarg_data = (double) ((int) (vtarg_set_val*100 + .5))/100;
           if(sscanf_success < 1 || vtarg_set_val < 0) {
             pmsg_error("invalid value in -x %s\n", extended_param);
             rv = -1;
             break;
           }
+          my.vtarg_data = (double) ((int) (vtarg_set_val*100 + .5))/100;
           my.vtarg_set = true;
           continue;
         }
@@ -1999,12 +1999,12 @@ static int stk500v2_jtag3_parseextparms(const PROGRAMMER *pgm, const LISTID extp
           double vtarg_set_val = 0;
           int sscanf_success = sscanf(extended_param, "vtarg=%lf", &vtarg_set_val);
 
-          my.vtarg_data = (double) ((int) (vtarg_set_val*100 + .5))/100;
           if(sscanf_success < 1 || vtarg_set_val < 0) {
             pmsg_error("invalid value in -x %s\n", extended_param);
             rv = -1;
             break;
           }
+          my.vtarg_data = (double) ((int) (vtarg_set_val*100 + .5))/100;
           my.vtarg_set = true;
           continue;
         }