Fix: msdms log fields emit dash instead of -1 for unset milestones

include/proxy/logging/LogAccess.h

@@ -316,6 +316,7 @@ class LogAccess
   static int     unmarshal_itox(int64_t val, char *dest, int field_width = 0, char leading_char = ' ');
   static int     unmarshal_int_to_str(char **buf, char *dest, int len);
   static int     unmarshal_int_to_str_hex(char **buf, char *dest, int len);
+  static int     unmarshal_milestone_diff(char **buf, char *dest, int len);
   static int     unmarshal_str(char **buf, char *dest, int len, LogSlice *slice, LogEscapeType escape_type);
   static int     unmarshal_ttmsf(char **buf, char *dest, int len);
   static int     unmarshal_int_to_date_str(char **buf, char *dest, int len);

src/proxy/logging/LogAccess.cc

@@ -626,6 +626,41 @@ LogAccess::unmarshal_int_to_str(char **buf, char *dest, int len)
   return -1;
 }
 
+/*-------------------------------------------------------------------------
+  LogAccess::unmarshal_milestone_diff
+
+  Unmarshal a milestone difference value.  Returns "-" when the
+  marshalled value is negative (milestone was unset), otherwise
+  the decimal string representation.
+  -------------------------------------------------------------------------*/
+
+int
+LogAccess::unmarshal_milestone_diff(char **buf, char *dest, int len)
+{
+  ink_assert(buf != nullptr);
+  ink_assert(*buf != nullptr);
+  ink_assert(dest != nullptr);
+
+  int64_t val = unmarshal_int(buf);
+  if (val < 0) {
+    if (len >= 1) {
+      dest[0] = '-';
+      return 1;
+    }
+    DBG_UNMARSHAL_DEST_OVERRUN
+    return -1;
+  }
+
+  char val_buf[128];
+  int  val_len = unmarshal_itoa(val, val_buf + 127);
+  if (val_len < len) {
+    memcpy(dest, val_buf + 128 - val_len, val_len);
+    return val_len;
+  }
+  DBG_UNMARSHAL_DEST_OVERRUN
+  return -1;
+}
+
 /*-------------------------------------------------------------------------
   LogAccess::unmarshal_int_to_str_hex
 

src/proxy/logging/LogField.cc

@@ -388,7 +388,7 @@ LogField::LogField(const char *field, Container container)
     if (0 != rv) {
       Note("Invalid milestone range in LogField ctor: %s", m_name);
     }
-    m_unmarshal_func = &(LogAccess::unmarshal_int_to_str);
+    m_unmarshal_func = &(LogAccess::unmarshal_milestone_diff);
     m_type           = LogField::sINT;
     break;
   }