goto to a cleanup/error handling block at the end of a function is a fairly common C idiom

Don't really see a problem using goto there. Simplyfies control flow and error handling. Assuming we are talking about same piece of code.

I think if you read the kennel's code you're in to a big surprise.

just because something's old doesn't mean it's bad