]> Kevux Git Server - fll/commitdiff
Bugfix: invalid number check in FSS Extended List write program.
authorKevin Day <thekevinday@gmail.com>
Thu, 15 Oct 2020 04:47:17 +0000 (23:47 -0500)
committerKevin Day <thekevinday@gmail.com>
Thu, 15 Oct 2020 04:47:17 +0000 (23:47 -0500)
The range is inclusive and as such the stop point is not the same as the string size.
Instead, it should be the string size minus one.

level_3/fss_extended_list_write/c/private-fss_extended_list_write.c

index 171f8481546f6587341232dbe2a44793e770bf4c..4fcfc11fa64b664917b63aa83fda0fa628d9fbce 100644 (file)
@@ -364,7 +364,7 @@ extern "C" {
       index = data.parameters[fss_extended_list_write_parameter_ignore].additional.array[i * 2];
 
       range.start = 0;
-      range.stop = strnlen(arguments.argv[index], f_console_length_size);
+      range.stop = strnlen(arguments.argv[index], f_console_length_size) - 1;
 
       // allow and ignore the positive sign.
       if (range.stop > 0 && arguments.argv[index][0] == '+') {
@@ -383,7 +383,7 @@ extern "C" {
       index = data.parameters[fss_extended_list_write_parameter_ignore].additional.array[(i * 2) + 1];
 
       range.start = 0;
-      range.stop = strnlen(arguments.argv[index], f_console_length_size);
+      range.stop = strnlen(arguments.argv[index], f_console_length_size) - 1;
 
       // allow and ignore the positive sign.
       if (range.stop > 0 && arguments.argv[index][0] == '+') {