Re: [PATCH v2 2/2] test: Fix issues reported by cppcheck static analysis tool
Bryce Harrington <[email protected]>
| Newsgroups | gmane.comp.lib.cairo |
|---|---|
| Message-ID | <[email protected]> |
On Fri, Aug 22, 2014 at 11:45:34AM +0530, RAVI NANJUNDAPPA wrote: > > > > -----Original Message----- > > From: cairo [mailto:[email protected]] On Behalf Of Bertram > > Felgenhauer > > Sent: Friday, August 22, 2014 10:58 AM > > To: [email protected] > > Subject: Re: [cairo] [PATCH v2 2/2] test: Fix issues reported by cppcheck > > static analysis tool > > > > 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? > > Sure. Not a problem. > But am not sure of why in this it is written like this, because usually > callee knows whether it has allocated the memory to data or not. > As of now, just for retaining the history, I haven't modified it. I hope > that should be fine. Since we're tinkering with the test already, may as well fix it up completely. Sounds like we all basically agree it should handle freeing its memory itself on error, can you reroll the patch to do that too? Bryce > Thanks and Best Regards, > N Ravi > > > > Cheers, > > > > Bertram > > -- > > cairo mailing list > > [email protected] > > http://lists.cairographics.org/mailman/listinfo/cairo > > -- > cairo mailing list > [email protected] > http://lists.cairographics.org/mailman/listinfo/cairo -- cairo mailing list [email protected] http://lists.cairographics.org/mailman/listinfo/cairo