...now this code needs refactoring.

We goto done at the end of these blocks.

This assignment isn't doing anything, so it safe to remove it, but
there remains the question of whether there could have been some
actually necessary intention behind this line.

execv(3) only returns on error, and will always return -1 in that
case.

Fix a use after free in a 'theoretical' case in switch_cache_db_persistant_execute_trans_full where the pre_trans_execute statement fails, but SQLSetAutoCommitAttr succeeds

we always have the caller profile at this point, if for some reason we don't just instead of crashing