Update libmseed to v3.1.1

Author: chad-earthscope

ChangeLog

@@ -1,3 +1,6 @@
+2024.024: 4.2
+	- Update libmseed to v3.1.1.
+
 2024.006: 4.1
 	- Update libmseed to v3.0.18.
 

libmseed/ChangeLog

@@ -1,3 +1,27 @@
+2024.024: 3.1.1
+	- Change library compatibility version in Makefile to MAJOR.1.0, as this is now
+	incompatible with the x.0.0 releases.
+
+2024.024: 3.1.0
+	BREAKING CHANGES, data structure and an API changes:
+	- `MS3Record.encoding` now int16_t instead of int8_t.
+	- `MS3Record.datalength` now uint32_t instead of uint16_t (change in 3.0.18).
+	- `MS3Record.datasize` and `MS3TraceSeg.datasize` now uint64_t instead of size_t.
+	- `ms3_detect()` now returns int64_t instead of int.
+	- `mstl3_unpack_recordlist()`'s `outputsize` argument is now uint64_t instead of size_t.
+	- `mseh_get_ptr_r()`'s `maxlength` is now uint32_t instead of size_t.
+
+	- Fix to handle full range of allowed encoding values from 0-255.
+	- Fix to handle detection of huge record lengths beyond MAXRECLEN gracefully.
+	- Add more checks to avoid writing impossible values to miniSEED v2 blockette fields.
+	- Fix all compiler warnings at level -Wextra for clang, gcc, and \W3 for MSVC.
+
+2024.015:
+	- Fix build of DLL target on Windows by defining exports in libmseed.def.
+
+2024.007:
+	- Improve formatting of gap list produced by mstl3_printgaplist().
+
 2024.006: 3.0.18
 	- ms_nslc2sid() requires all codes except location to be set to reduce misuse.
 	- Fix raw, diagnostic generation of SIDs from miniSEED v2 when full codes are used.

libmseed/Makefile

@@ -9,7 +9,7 @@
 # Extract version from libmseed.h, expected line should include LIBMSEED_VERSION "#.#.#"
 MAJOR_VER = $(shell grep LIBMSEED_VERSION libmseed.h | grep -Eo '[0-9]+.[0-9]+.[0-9]+' | cut -d . -f 1)
 FULL_VER = $(shell grep LIBMSEED_VERSION libmseed.h | grep -Eo '[0-9]+.[0-9]+.[0-9]+')
-COMPAT_VER = $(MAJOR_VER).0.0
+COMPAT_VER = $(MAJOR_VER).1.0
 
 # Default settings for install target
 PREFIX ?= /usr/local

libmseed/Makefile.win

@@ -34,7 +34,7 @@ lib: $(OBJS)
 	link.exe /lib /nologo /OUT:$(LM_LIB) $(OBJS)
 
 dll: $(OBJS)
-	link.exe /dll /nologo /OUT:$(LM_DLL) $(OBJS)
+	link.exe /dll /nologo /DEF:$(LM_DEF) /OUT:$(LM_DLL) $(OBJS)
 
 .c.obj:
 	$(CC) /nologo $(CFLAGS) $(INCS) $(OPTS) /c $<

libmseed/example/Makefile.win

@@ -9,7 +9,7 @@ OPTS = /O2 /D_CRT_SECURE_NO_WARNINGS
 SRCS = lm_pack.c \
        lm_pack_rollingbuffer.c \
        lm_parse.c \
-	   lm_read_buffer.c \
+       lm_read_buffer.c \
        lm_read_recordlist.c \
        lm_read_selection.c \
        lm_sids.c \
@@ -25,4 +25,5 @@ all: $(BINS)
 
 # Clean-up directives
 clean:
-	-del *.obj *.exe *% *~
+	-del *.obj *.exe *% *~
+

libmseed/example/lm_pack_rollingbuffer.c

@@ -37,6 +37,7 @@
 void
 record_handler (char *record, int reclen, void *handlerdata)
 {
+  (void)handlerdata;
   MS3Record *msr = NULL;
 
   if (!msr3_parse (record, reclen, &msr, 0, 0))

libmseed/example/lm_read_recordlist.c

@@ -41,15 +41,15 @@ main (int argc, char **argv)
   char fileptrstr[30];
   uint32_t flags = 0;
   int8_t verbose = 0;
-  size_t idx;
+  uint64_t idx;
   int rv;
 
   char printdata = 0;
   int64_t unpacked;
   uint8_t samplesize;
   char sampletype;
-  size_t lineidx;
-  size_t lines;
+  uint64_t lineidx;
+  uint64_t lines;
   int col;
   void *sptr;
 
@@ -62,7 +62,7 @@ main (int argc, char **argv)
   mseedfile = argv[1];
 
   /* Simplistic argument parsing */
-  for (idx = 2; idx < argc; idx++)
+  for (idx = 2; (int)idx < argc; idx++)
   {
     if (strncmp (argv[idx], "-v", 2) == 0)
       verbose += strspn (&argv[idx][1], "v");
@@ -166,7 +166,7 @@ main (int argc, char **argv)
 
             for (idx = 0, lineidx = 0; lineidx < lines; lineidx++)
             {
-              for (col = 0; col < 6 && idx < seg->numsamples; col++)
+              for (col = 0; col < 6 && (int64_t)idx < seg->numsamples; col++)
               {
                 sptr = (char *)seg->datasamples + (idx * samplesize);
 

libmseed/extraheaders.c

@@ -148,7 +148,7 @@ parse_json (char *jsonstring, size_t length, LM_PARSED_JSON *parsed)
  ***************************************************************************/
 int
 mseh_get_ptr_r (const MS3Record *msr, const char *ptr,
-                 void *value, char type, size_t maxlength,
+                 void *value, char type, uint32_t maxlength,
                  LM_PARSED_JSON **parsestate)
 {
   LM_PARSED_JSON *parsed  = (parsestate) ? *parsestate : NULL;

libmseed/fileutils.c

@@ -768,6 +768,8 @@ int
 ms3_url_useragent (const char *program, const char *version)
 {
 #if !defined(LIBMSEED_URL)
+  (void)program; /* Unused */
+  (void)version; /* Unused */
   ms_log (2, "URL support not included in library\n");
   return -1;
 #else
@@ -796,6 +798,7 @@ int
 ms3_url_userpassword (const char *userpassword)
 {
 #if !defined(LIBMSEED_URL)
+  (void) userpassword; /* Unused */
   ms_log (2, "URL support not included in library\n");
   return -1;
 #else
@@ -824,6 +827,7 @@ int
 ms3_url_addheader (const char *header)
 {
 #if !defined(LIBMSEED_URL)
+  (void)header; /* Unused */
   ms_log (2, "URL support not included in library\n");
   return -1;
 #else
@@ -1008,12 +1012,12 @@ mstl3_writemseed (MS3TraceList *mstl, const char *mspath, int8_t overwrite,
 char *
 parse_pathname_range (const char *string, int64_t *start, int64_t *end)
 {
-  char startstr[21] = {0}; /* Maximum of 20 digit value */
-  char endstr[21]   = {0}; /* Maximum of 20 digit value */
-  int startdigits   = 0;
-  int enddigits     = 0;
-  char *dash        = NULL;
-  char *at          = NULL;
+  char startstr[21]   = {0}; /* Maximum of 20 digit value */
+  char endstr[21]     = {0}; /* Maximum of 20 digit value */
+  uint8_t startdigits = 0;
+  uint8_t enddigits   = 0;
+  char *dash          = NULL;
+  char *at            = NULL;
   char *ptr;
 
   if (!string || (!start || !end))

libmseed/genutils.c

@@ -139,7 +139,7 @@ static const int monthdays_leap[] = {31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30,
 #define VALIDSEC(sec) (sec >= 0 && sec <= 60)
 
 /* Check that a nanosecond is in a valid range */
-#define VALIDNANOSEC(nanosec) (nanosec >= 0 && nanosec <= 999999999)
+#define VALIDNANOSEC(nanosec) (nanosec <= 999999999)
 
 /** @endcond */
 
@@ -314,6 +314,7 @@ int
 ms_nslc2sid (char *sid, int sidlen, uint16_t flags,
              const char *net, const char *sta, const char *loc, const char *chan)
 {
+  (void)flags; /* Unused */
   char *sptr = sid;
   char xchan[6] = {0};
   int needed = 0;
@@ -411,7 +412,7 @@ ms_nslc2sid (char *sid, int sidlen, uint16_t flags,
     return -1;
   }
 
-  return (sptr - sid);
+  return (int)(sptr - sid);
 } /* End of ms_nslc2sid() */
 
 /**********************************************************************/ /**
@@ -1344,7 +1345,7 @@ ms_timestr2nstime (const char *timestr)
     }
   }
 
-  length = cp - timestr;
+  length = (int)(cp - timestr);
 
   /* If the time string is all number-like characters assume it is an epoch time.
    * Unless it is 4 characters, which could be a year, unless it starts with a sign. */
@@ -1431,15 +1432,15 @@ nstime_t
 ms_mdtimestr2nstime (const char *timestr)
 {
   int fields;
-  int year    = 0;
-  int mon     = 1;
-  int mday    = 1;
-  int yday    = 1;
-  int hour    = 0;
-  int min     = 0;
-  int sec     = 0;
-  double fsec = 0.0;
-  int nsec    = 0;
+  int year      = 0;
+  int mon       = 1;
+  int mday      = 1;
+  int yday      = 1;
+  int hour      = 0;
+  int min       = 0;
+  int sec       = 0;
+  double fsec   = 0.0;
+  uint32_t nsec = 0;
 
   if (!timestr)
   {
@@ -1453,7 +1454,7 @@ ms_mdtimestr2nstime (const char *timestr)
   /* Convert fractional seconds to nanoseconds */
   if (fsec != 0.0)
   {
-    nsec = (int)(fsec * 1000000000.0 + 0.5);
+    nsec = (uint32_t)(fsec * 1000000000.0 + 0.5);
   }
 
   if (fields < 1)
@@ -1500,7 +1501,7 @@ ms_mdtimestr2nstime (const char *timestr)
 
   if (!VALIDNANOSEC (nsec))
   {
-    ms_log (2, "fractional second (%d) is out of range\n", nsec);
+    ms_log (2, "fractional second (%u) is out of range\n", nsec);
     return NSTERROR;
   }
 
@@ -1539,13 +1540,13 @@ nstime_t
 ms_seedtimestr2nstime (const char *seedtimestr)
 {
   int fields;
-  int year    = 0;
-  int yday    = 1;
-  int hour    = 0;
-  int min     = 0;
-  int sec     = 0;
-  double fsec = 0.0;
-  int nsec    = 0;
+  int year      = 0;
+  int yday      = 1;
+  int hour      = 0;
+  int min       = 0;
+  int sec       = 0;
+  double fsec   = 0.0;
+  uint32_t nsec = 0;
 
   if (!seedtimestr)
   {
@@ -1559,7 +1560,7 @@ ms_seedtimestr2nstime (const char *seedtimestr)
   /* Convert fractional seconds to nanoseconds */
   if (fsec != 0.0)
   {
-    nsec = (int)(fsec * 1000000000.0 + 0.5);
+    nsec = (uint32_t)(fsec * 1000000000.0 + 0.5);
   }
 
   if (fields < 1)
@@ -1600,7 +1601,7 @@ ms_seedtimestr2nstime (const char *seedtimestr)
 
   if (!VALIDNANOSEC (nsec))
   {
-    ms_log (2, "fractional second (%d) is out of range\n", nsec);
+    ms_log (2, "fractional second (%u) is out of range\n", nsec);
     return NSTERROR;
   }