Hello, > - free(ptl_mait->countries[i].pf_ptl_mai); > + if (ptl_mait->countries[i].pf_ptl_mai) > + free(ptl_mait->countries[i].pf_ptl_mai); AFAIK free(NULL) is totally valid so this change is just a non-op. Best regards -- Rémi Duraffort | ivoire http://ivoire.dinauz.org/blog/