Fix double release in error handling
* src/mail.cpp (fixup_last_attachment): Do not release attach on error.
The attach object is released at the end of the function, this
would have resulted in a double release. There is no known
issue where this occurs since we don't really do content-ids
currently. But noticed this while looking for a different refcount