tree fd400d19de53e6a25ffc5c78603924f4d88037c1
parent 31c6ce3b63f8a494ad9e31ca65187a73d8ad3508
author Elliott Hughes <enh@google.com> 1603823360 -0700
committer Nick Wellnhofer <wellnhofer@aevum.de> 1604942221 +0100

Fix xmlURIEscape memory leaks.

Found by running the fuzz/uri.c fuzzer under asan (internal Android bug
171610679).

Always free `ret` when exiting on failure. I've moved the definition of
NULLCHK down past where ret is always initialized to make it clear that
this is safe.

This patch also fixes the indentation of two of the NULLCHK call sites
to make it more obvious that NULLCHK isn't `if`-like.