Remove unnecessary code after drupal_goto() that will never execute

As per function drupal_goto()

This function ends the request; use it instead of a return in your menu callback.

It's confusing that there are many cases where drupal_goto() is called in this module and afterwards you have exit; return;, this kind of stuff. Anything after drupal_goto() will never run since drupal_exit() is called on the last line of the function.