Sanity checking a reference to ensure it's still valid before using it is always a good idea. Just because the bug originated elsewhere doesn't make it a good idea to blindly assume attackby() is still referencing an object. The /mob/attackby() proc does the same thing.

However this is all a moot point as I just realized an issue with this that needs fixing anyways.