That's a safe use of exec, since the name is coming from a loaded module and not from user input.  Still, load_module would be cleaner.

However, the real fix is to just delete that code.  It is dead code...we always pass the indirect_test argument to dash_R.