mapwcs20: keep compiler happy about (im)possible use of uninitialized…

… memory
MapServer · Aug 20, 2018 · 0bf81bb · 0bf81bb
1 parent d18302b
commit 0bf81bb
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/mapwcs20.c b/mapwcs20.c
@@ -2806,7 +2806,6 @@ static int msWCSGetCoverageMetadata20(layerObj *layer, wcs20coverageMetadataObj
       /* set default values for interval and significant figures */
       switch(cm->imagemode) {
         case MS_IMAGEMODE_BYTE:
-        case MS_IMAGEMODE_PC256:
           default_values.interval_min = 0.;
           default_values.interval_max = 255.;
           default_values.significant_figures = 3;
@@ -2821,6 +2820,11 @@ static int msWCSGetCoverageMetadata20(layerObj *layer, wcs20coverageMetadataObj
           default_values.interval_max = FLT_MAX;
           default_values.significant_figures = 12;
           break;
+        default:
+          // other imagemodes are invalid (see above), just keep the compiler happy
+          msFreeCharArray(band_names, num_band_names);
+          return MS_FAILURE;
+          break;
       }
 
       /* lookup default interval values */