Frantisek Dvorak has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2687

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/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2687
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I71e5bb754866003e2a282ce3eeb8d4e6eac0573d
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Frantisek Dvorak <val...@civ.zcu.cz>

Reply via email to