Skip to content

Commit

Permalink
Remove redundant message in AddUserToTokenDacl().
Browse files Browse the repository at this point in the history
GetTokenUser() will have reported an adequate error message.  These
error conditions almost can't happen, so users are unlikely to observe
this change.

Reviewed by Tom Lane and Stephen Frost.
  • Loading branch information
nmisch committed Apr 7, 2016
1 parent 29dd150 commit 33d3fc5
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions src/common/exec.c
Original file line number Diff line number Diff line change
Expand Up @@ -674,10 +674,7 @@ AddUserToTokenDacl(HANDLE hToken)

/* Get the current user SID */
if (!GetTokenUser(hToken, &pTokenUser))
{
log_error("could not get token user: error code %lu", GetLastError());
goto cleanup;
}
goto cleanup; /* callee printed a message */

/* Figure out the size of the new ACL */
dwNewAclSize = asi.AclBytesInUse + sizeof(ACCESS_ALLOWED_ACE) +
Expand Down

0 comments on commit 33d3fc5

Please sign in to comment.