On Sat, Sep 08, 2018 at 07:56:23AM -0400, Tom Lane wrote: > Looking closer, there are no leaks, though I think we could use a comment > about that. And it still makes me itch that pg_saslprep (mostly) doesn't > worry about clearing *output on failure while only three of its four > callers bother to initialize their variables to null. That's a recipe > for future bugs. So I propose the attached cleanup.
Okay, that looks acceptable to me. Thanks for looking at that. -- Michael