Going back to strcpy instead of memset

AbelPau · Apr 17, 2024 · e11c902 · e11c902
1 parent e88acdc
commit e11c902
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/ogr/ogrsf_frmts/miramon/mm_gdal_functions.c b/ogr/ogrsf_frmts/miramon/mm_gdal_functions.c
@@ -2234,7 +2234,6 @@ int MM_SprintfDoubleSignifFigures(char *szChain, size_t size_szChain,
         return 0;
     exponent = atoi(ptr + 1);
 
-    dfRealValue = CPLAtof(szChain);
     return sprintf(szChain, "%.*f",
                    (nSignifFigures - exponent - 1) > 0
                        ? (nSignifFigures - exponent - 1)
@@ -2408,9 +2407,7 @@ int MM_SecureCopyStringFieldValue(char **pszStringDst, const char *pszStringSrc,
         *nStringCurrentLength = (MM_EXT_DBF_N_FIELDS)(strlen(pszStringSrc) + 1);
     }
 
-    // We just copy the exact number of elements we want to copy.
-    // If we copy more than them, some garbage can appear in the record
-    memcpy(*pszStringDst, pszStringSrc, strlen(pszStringSrc));
+    strcpy(*pszStringDst, pszStringSrc);
     return 0;
 }