Henry Robinson has uploaded a new patch set (#2). Change subject: IMPALA-2645: Set mechUsing only on success ......................................................................
IMPALA-2645: Set mechUsing only on success Better to check success status from sasl_client_start() before using returned structure with auth mechanism. According to the API documentation, its value is guaranteed only after success (no error, or SASL_CONTINUE). Change-Id: I71e5bb754866003e2a282ce3eeb8d4e6eac0573d --- M be/src/transport/TSasl.cpp 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/87/2687/2 -- To view, visit http://gerrit.cloudera.org:8080/2687 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I71e5bb754866003e2a282ce3eeb8d4e6eac0573d Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Frantisek Dvorak <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]>
