1. Your title is very not specific about the problem
2. You can use the [#] icon to insert CODE tags and paste the macro between them -- I did it for you this time
3. Seems more of an Outlook question and maybe the Outlook gurus can help - let me know if you want it moved
4. Try running this and see how Outlook wants to name the file
Sub saveAttachtoDisk(itm As Outlook.MailItem)
Dim objAtt As Outlook.Attachment
Dim saveFolder As String
saveFolder = "Y:\accounts\success fee bills"
For Each objAtt In itm.Attachments
objAtt.SaveAsFile saveFolder & "" & itm.Subject & ".XLS"
MsgBox saveFolder & "\" & itm.Subject & ".XLS" ' I added the \
Next
End Sub