commit | 7c06d99e1f4f853e3c5b307c0dc79c8a32a09855 | [log] [tgz] |
---|---|---|
author | Elliott Hughes <enh@google.com> | Tue Oct 27 11:29:20 2020 -0700 |
committer | Nick Wellnhofer <wellnhofer@aevum.de> | Mon Nov 09 18:17:01 2020 +0100 |
tree | fd400d19de53e6a25ffc5c78603924f4d88037c1 | |
parent | 31c6ce3b63f8a494ad9e31ca65187a73d8ad3508 [diff] |
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.