From b978043f263878aa1b063c37e1939819174a00cc Mon Sep 17 00:00:00 2001
From: Wouter van Gulik <pa3wle@gmail.com>
Date: Sat, 11 Jul 2020 21:45:59 +0200
Subject: [PATCH] Fix two bugs regarding the shown marker frequency: The
 function frequency_string_short did not append 'Mhz' properly for 1-9Mhz
 cell_draw_marker_info stored frequency in a int32_t instead of freqHz_t,
 which caused sign conversion after 2.1Ghz, making marker reading beyond that
 point bogus. Note that this is only visible with multiple markers.

---
 plot.cpp | 44 +++++++++++++++++++++++++++++++++++---------
 1 file changed, 35 insertions(+), 9 deletions(-)

diff --git a/plot.cpp b/plot.cpp
index e7cd74d..09a4f81 100644
--- a/plot.cpp
+++ b/plot.cpp
@@ -97,7 +97,7 @@ void update_grid(void)
 		fspan = -frequency1;
 		fstart = frequency0 - fspan/2;
 	}
-	
+
 	while (gdigit > 100) {
 		grid = 5 * gdigit;
 		if (fspan / grid >= 4)
@@ -229,7 +229,7 @@ smith_grid2(int x, int y, float scale)
 	// offset to center
 	x -= P_CENTER_X;
 	y -= P_CENTER_Y;
-	
+
 	// outer circle
 	d = circle_inout(x, y, P_RADIUS);
 	if (d < 0)
@@ -325,7 +325,7 @@ smith_grid3(int x, int y)
 	// offset to center
 	x -= P_CENTER_X;
 	y -= P_CENTER_Y;
-	
+
 	// outer circle
 	d = circle_inout(x, y, P_RADIUS);
 	if (d < 0)
@@ -1595,7 +1595,7 @@ cell_draw_marker_info(int x0, int y0)
 			cell_drawstring(buf, xpos, ypos);
 			xpos += 3*FONT_WIDTH + 3;
 			//trace_get_info(t, buf, sizeof buf);
-			int32_t freq = freqAt(markers[mk].index);
+			freqHz_t freq = freqAt(markers[mk].index);
 			if (uistat.marker_delta && mk != active_marker) {
 				freq -= freqAt(markers[active_marker].index);
 				frequency_string_short(buf, sizeof buf, freq, S_DELTA[0]);
@@ -1691,6 +1691,11 @@ cell_draw_marker_info(int x0, int y0)
 	}
 }
 
+/* Prints a full frequency:
+ * 500.000 Khz
+ * 10.000 000 Mhz
+ * 3000.000 000 Mhz
+ */
 void
 frequency_string(char *buf, size_t len, freqHz_t freq)
 {
@@ -1713,6 +1718,11 @@ frequency_string(char *buf, size_t len, freqHz_t freq)
 	}
 }
 
+/* Prints a shorter/compacter frequency:
+ * 500.000KHz
+ * 10.000000Mhz
+ * 3000.0000Mhz
+ */
 void
 frequency_string_short(char *b, size_t len, freqHz_t freq, char prefix)
 {
@@ -1732,12 +1742,28 @@ frequency_string_short(char *b, size_t len, freqHz_t freq, char prefix)
 		chsnprintf(buf, len, "%d.%03dkHz",
 						 (int)(freq / 1000),
 						 (int)(freq % 1000));
-	} else {
-		chsnprintf(buf, len, "%d.%06d",
-						 (int)(freq / 1000000),
-						 (int)(freq % 1000000));
-		strcpy(b+9, "MHz");
 	}
+	size_t wr = chsnprintf(buf, len, "%d.%06d",
+			(int)(freq / 1000000),
+			(int)(freq % 1000000));
+	// Note that wr is either:
+	// 8: 1 - 9 Mhz
+	// 9: 10 - 99 Mhz
+	// 10: 100 - 999 Mhz
+	// 11: 1000 - 4400 Mhz
+	// Do not add 'Mhz' if it would overwrite all characters
+	if (len < 4)
+		return;
+	// Overwrite last digits, if >= 10Mhz (aka 9 digits)
+	if (wr > 9) {
+		wr = 9;
+	}
+	// Make sure we have space left for 'Mhz'
+	if (wr > len - 4) {
+		wr = len - 4;
+	}
+
+	strcpy(buf + wr, "MHz");
 }
 
 void padString(char* s, int len, char c = ' ') {