degrib: fix cppcheck nullPointerOutOfMemory

OSGeo · Dec 16, 2024 · 378cda2 · 378cda2
1 parent 8bea7c7
commit 378cda2
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/frmts/grib/degrib/degrib/degrib1.cpp b/frmts/grib/degrib/degrib/degrib1.cpp
@@ -834,6 +834,11 @@ int GRIB1_RefTime (VSILFILE *fp, uInt4 gribLen, double *refTime)
       return -1;
    }
    pds = (uChar *) malloc (sectLen * sizeof (uChar));
+   if(!pds)
+   {
+       errSprintf("Out of memory");
+       return -1;
+   }
    *pds = *temp;
    pds[1] = temp[1];
    pds[2] = temp[2];

diff --git a/frmts/grib/degrib/degrib/metaparse.cpp b/frmts/grib/degrib/degrib/metaparse.cpp
@@ -475,6 +475,10 @@ static int ParseSect2_Wx (float *rdat, sInt4 nrdat, sInt4 *idat,
 
    buffLen = 0;
    buffer = (char *) malloc ((nidat + 1) * sizeof (char));
+   if( !buffer ) {
+       errSprintf ("Out of memory\n");
+       return -1;
+   }
    while (groupLen > 0) {
       for (j = 0; j < groupLen; j++) {
          buffer[buffLen] = (char) idat[loc];
@@ -600,6 +604,10 @@ static int ParseSect2_Hazard (float *rdat, sInt4 nrdat, sInt4 *idat,
 
    buffLen = 0;
    buffer = (char *) malloc ((nidat + 1) * sizeof (char));
+   if (!buffer) {
+       errSprintf ("Out of memory\n");
+       return -1;
+   }
    while (groupLen > 0) {
       for (j = 0; j < groupLen; j++) {
          buffer[buffLen] = (char) idat[loc];