Hacker News Reader

Yes, if you know that the process is going to exit. In this case, the missing free is in a function which is about a dozen calls down in the stack, and I know that it's harmless right now because I've looked at the code and verified that an error there will always result in tarsnap exiting; but if I don't fix this now it's quite possible that in the future I'll have code which uses that function but can tolerate it failing.

View on HN

I think I understand. Function, which could be used independently should free all used resources.

If a function is very project-specific and has no purpose to being used independently it may be worth considering declaring it static, right?

"static" is a different concept -- that marks a function as not being used from outside of that compilation unit (aka .c file). This particular function was used from outside of that file, but was unlikely to be reused in a different project.

Reply on news.ycombinator.com