On Fri, Jun 26, 2020 at 09:17:36AM -0700, Kevin J. McCarthy wrote:
One thing I can do is keep the escaping *only* for the cases it might be needed, such as hook command extraction.
FYI, after reviewing, the only things I can see that are run through mutt_extract_token() twice are hook command arguments.
I've just pushed two commits. One converting var_to_string() to use a buffer and remove the built-in escaping. The other to re-enable escaping just for those hooks command argument.
Please let me know if you encounter any problems from this change. Thank you! -- Kevin J. McCarthy GPG Fingerprint: 8975 A9B3 3AA3 7910 385C 5308 ADEF 7684 8031 6BDA
signature.asc
Description: PGP signature
