]> Kevux Git Server - fll/commitdiff
Bugfix: Improper if condition range check.
authorKevin Day <thekevinday@gmail.com>
Sat, 26 Feb 2022 03:45:37 +0000 (21:45 -0600)
committerKevin Day <thekevinday@gmail.com>
Sat, 26 Feb 2022 03:45:37 +0000 (21:45 -0600)
This is supposed to check the start and stop range, but accidentally checks the start range on both side.
This mistake appears to have been copy pasted quite a few times.

level_1/fl_print/c/private-print.c

index 983b1b8f9a51aee54fb1be80500761f1e255bcd0..e4af7059432dfed82c7bc2d9260f29a3337071e3 100644 (file)
@@ -270,7 +270,7 @@ extern "C" {
                 except_in = va_arg(*ap, f_string_ranges_t);
               }
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -293,7 +293,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = va_arg(*ap, f_string_ranges_t);
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -316,7 +316,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = f_string_ranges_t_initialize;
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -387,7 +387,7 @@ extern "C" {
                 except_in = va_arg(*ap, f_string_ranges_t);
               }
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -410,7 +410,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = va_arg(*ap, f_string_ranges_t);
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -433,7 +433,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = f_string_ranges_t_initialize;
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -668,7 +668,7 @@ extern "C" {
                 except_in = va_arg(*ap, f_string_ranges_t);
               }
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -691,7 +691,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = va_arg(*ap, f_string_ranges_t);
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -714,7 +714,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = va_arg(*ap, f_string_ranges_t);
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -792,7 +792,7 @@ extern "C" {
                 except_in = va_arg(*ap, f_string_ranges_t);
               }
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -815,7 +815,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = va_arg(*ap, f_string_ranges_t);
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;
@@ -838,7 +838,7 @@ extern "C" {
               const f_array_lengths_t except_at = f_array_lengths_t_initialize;
               const f_string_ranges_t except_in = va_arg(*ap, f_string_ranges_t);
 
-              if (partial.start > partial.start) {
+              if (partial.start > partial.stop) {
                 *status = F_data_not;
 
                 return string;