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
issue.