The code looks ok. Try changing the trigger for the email to use the button press instead of current form submit trigger.
The attachment code looks wrong. It is just referencing the name of the attachment and not the location where it is saved. That could be an issue as well.