On 22.02.2013, at 21:36, Devon Yoo <9tontr...@gmail.com> wrote:

>       char* icc_1 = (char*)malloc(sizeof(char)*len_icc1 +1);
>       char* icc_2 = (char*)malloc(sizeof(char)*len_icc2 +1);

nil checks after malloc?

>       hInProfile = cmsOpenProfileFromFile(icc_2, "r");
>       hOutProfile = cmsOpenProfileFromFile(icc_1, "r");

nil checks after opening profile?

>       hTransform = cmsCreateTransform(hInProfile,
>                                                                       
> TYPE_BGR_8,
>                                                                       
> hOutProfile,
>                                                                       
> TYPE_BGR_8,
>                                                                       
> INTENT_PERCEPTUAL, 0);

Nil check after transform?

>       free(icc_1);
>       free(icc_2);

You need to verify that results are valid!

Greetings
Christian

-- 
Read our blog about news on our plugins:

http://www.mbsplugins.de/


------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_feb
_______________________________________________
Lcms-user mailing list
Lcms-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lcms-user

Reply via email to