This is one of the strange VB problems you may face using even very simple macros.
Your macro runs fine, but in the end (end sub) you receive a code execution error, that the macro is interrupted and you need to debug, etc.
Here is a way to avoid this absurd problem.
In the beginning of your macro, before all other code lines, add this one:
Application.EnableCancelKey = xlDisabled
Enjoy!