[cairo] [PATCH v2 2/2] test: Fix issues reported by cppcheck static analysis tool

Bertram Felgenhauer bertram.felgenhauer at googlemail.com
Thu Aug 21 22:27:39 PDT 2014


RAVI NANJUNDAPPA wrote:
> Hello Bertram, 
>  
> If you look at the pdf-mime-data.c file, 'free(data)' is handled by
> the callee at line numbers (from original file): 108, 153 and 159.

Sorry, I had not looked at the callers. It's just unusual that for a
failing function call to leave such cleanup to its callers. Is there
any reason for doing so in this case?

Cheers,

Bertram


More information about the cairo mailing list