]> granicus.if.org Git - neomutt/commit
Remove two use-after free in global hooks (#353)
authorguiniol <gui-gui@netcourrier.com>
Wed, 1 Feb 2017 13:26:35 +0000 (14:26 +0100)
committerRichard Russon <rich@flatcap.org>
Wed, 1 Feb 2017 13:26:35 +0000 (13:26 +0000)
commit384726d556c8a040a66b41eaecd506e909508cab
tree79a0e3233eecd3cc98a94036b0804948e3e50622
parentd8566d5412e9890e27dcf51309b25f69436c8a3a
Remove two use-after free in global hooks (#353)

Fixes #347
In both mutt_timeout_hook and mutt_startup_shutdown_hook, we iterate on
the list of hooks and try to execute the right ones, always using the
same buffer. The buffer must be free'd after all the hooks have been
checked and not after the first hook which fails. This also removes a
memory leak, because the buffer was never freed if no hook failed.
hook.c