Fixes calling cleanup method when post/auxiliary modules fail

[EclipseAditya]

Fixes #18138

Post and Auxiliary modules were invoking mod.cleanup twice when exiting through error paths (fail_with, Timeout, Interrupt, or unhandled exceptions). Cleanup was first triggered inside the rescue blocks of job_run_proc and then executed again by job_cleanup_proc, which run_simple calls unconditionally.

Root cause: job_run_proc explicitly called mod.cleanup in each rescue clause, while run_simple always invoked job_cleanup_proc afterward, resulting in duplicate cleanup.

Resolution: Removed direct mod.cleanup calls from all rescue blocks in job_run_proc for Msf::Simple::Post and Msf::Simple::Auxiliary. The synchronous execution path is now wrapped in begin/ensure so job_cleanup_proc runs exactly once regardless of how execution exits. Applied the same fix in ExploitDriver, where an Interrupt rescue caused duplicate cleanup during multi-target runs.

Verification: Executed the updated test suite (24 examples), all passing:
bundle exec rspec spec/lib/msf/base/simple/

[msutovsky-r7]

So the fix here seems odd - it does not take into account some categories of modules, where the bug persists and for some, it removes calling cleanup entirely when module throw some specific error(?). Not really sure, I'll investigate more here.

[msutovsky-r7]

I think this issue requires little bit more testing and more time - I've opened new PR, but I would like to keep the specs from this PR.