[#2212] CID43789 and CID43788:

turned on doAccessControl by default... fairly sure we want to test that
[#2212] CID43787:

Removed ACDebug as it is always 0, and the debug messages it provided
were just checkpoints. If we need to debug this function, we can use
a debugger.
[#2212] CID37617:

Moved return to after assignment of out variables
[#2212] CID37614:

Removed vestigial error-checking code
[#2212] CID37610:

Removed vestigial code
[#2212] CID37599:

Removed generic error check in message in favor of the more specific one
found above it.
[#2212] CID37597:

Make logic more explicit, minor refactor to avoid scope creep