vlistInqVarLongname: return CDI_MAX_NAME chars (bug fix).

f0cf1a5c · Uwe Schulzweida · 59efbbdf · f0cf1a5c · f0cf1a5c
Commit f0cf1a5c authored May 17, 2019 by Uwe Schulzweida
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 2 deletions

ChangeLog ChangeLog +4 -0

src/vlist_var.c src/vlist_var.c +6 -2

No files found.
--- a/ChangeLog
+++ b/ChangeLog
@@ -3,6 +3,10 @@
        * using CGRIBEX library version 1.9.3
 	* Version 1.9.7 released

+2019-05-17  Uwe Schulzweida
+
+	* vlistInqVarLongname: return CDI_MAX_NAME chars (bug fix)
+
 2019-04-26  Uwe Schulzweida

 	* netCDF4: dimids are greater than ndims after ncrename (bug fix)

--- a/src/vlist_var.c
+++ b/src/vlist_var.c
@@ -579,7 +579,11 @@ void vlistInqVarLongname(int vlistID, int varID, char *longname)
 	}
    }
  else
-    strcpy(longname, vlistptr->vars[varID].longname);
+    {
+      // strcpy(longname, vlistptr->vars[varID].longname);
+      strncpy(longname, vlistptr->vars[varID].longname, CDI_MAX_NAME);
+      longname[CDI_MAX_NAME-1] = 0;
+    }

  return;
 }
@@ -716,7 +720,7 @@ The function @func{vlistInqVarDatatype} returns the data type of a variable.
 @Result
 @func{vlistInqVarDatatype} returns an identifier to the data type of the variable.
 The valid CDI data types are @func{CDI_DATATYPE_PACK8}, @func{CDI_DATATYPE_PACK16}, @func{CDI_DATATYPE_PACK24},
-@func{CDI_DATATYPE_FLT32}, @func{CDI_DATATYPE_FLT64}, @func{CDI_DATATYPE_INT8}, @func{CDI_DATATYPE_INT16} and 
+@func{CDI_DATATYPE_FLT32}, @func{CDI_DATATYPE_FLT64}, @func{CDI_DATATYPE_INT8}, @func{CDI_DATATYPE_INT16} and
 @func{CDI_DATATYPE_INT32}.

 @EndFunction