Merge pull request #406 from x2018/cmsFreeProfileSequenceDescription

optimize the code in cmsFreeProfileSequenceDescription() Ok, that's cosmetic but I like your attention to detail. Looks great, thank you!
mm2 · Oct 26, 2023 · 5c0655a · 5c0655a
2 parents 32ded9e + e5ec2b1
commit 5c0655a
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/cmsnamed.c b/src/cmsnamed.c
@@ -817,15 +817,16 @@ void CMSEXPORT cmsFreeProfileSequenceDescription(cmsSEQ* pseq)
     if (pseq == NULL)
         return;
 
-    for (i=0; i < pseq ->n; i++) {
-        if (pseq->seq != NULL) {
+    if (pseq ->seq != NULL) {
+        for (i=0; i < pseq ->n; i++) {
             if (pseq ->seq[i].Manufacturer != NULL) cmsMLUfree(pseq ->seq[i].Manufacturer);
             if (pseq ->seq[i].Model != NULL) cmsMLUfree(pseq ->seq[i].Model);
             if (pseq ->seq[i].Description != NULL) cmsMLUfree(pseq ->seq[i].Description);
         }
+
+        _cmsFree(pseq ->ContextID, pseq ->seq);
     }
 
-    if (pseq ->seq != NULL) _cmsFree(pseq ->ContextID, pseq ->seq);
     _cmsFree(pseq -> ContextID, pseq);
 }