Fix a double free in case of error

cmsReadTag already tracks its pointers so no need to free it
Also fixed a cosmetic warning (no functionality changes)

2 files changed, 4 insertions, 4 deletions

diff --git a/src/cmsio0.c b/src/cmsio0.c
index 821b869..222f7c7 100644
--- a/src/cmsio0.c
+++ b/src/cmsio0.c
@@ -742,9 +742,10 @@ cmsUInt32Number _validatedVersion(cmsUInt32Number DWord)
 static 
 cmsBool validDeviceClass(cmsProfileClassSignature cl)
 {
+    if ((int)cl == 0) return TRUE; // We allow zero because older lcms versions defaulted to that.
+
     switch (cl)
-    {
-    case 0: // We allow zero because older lcms versions defaulted to that.
+    {    
     case cmsSigInputClass:
     case cmsSigDisplayClass:
     case cmsSigOutputClass:
diff --git a/src/cmsio1.c b/src/cmsio1.c
index 4ef8671..2d7deee 100644
--- a/src/cmsio1.c
+++ b/src/cmsio1.c
@@ -737,8 +737,7 @@ cmsPipeline* CMSEXPORT _cmsReadDevicelinkLUT(cmsHPROFILE hProfile, cmsUInt32Numb
 
         return Lut;
     Error:
-        cmsPipelineFree(Lut);
-        cmsFreeNamedColorList(nc);
+        cmsPipelineFree(Lut);        
         return NULL;
     }