Skip to content

Commit

Permalink
Removed warning messages while compiling extensions
Browse files Browse the repository at this point in the history
  • Loading branch information
yitam committed Nov 30, 2018
1 parent cbdc01c commit 71570ca
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions source/shared/core_conn.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,7 @@ sqlsrv_conn* core_sqlsrv_connect( _In_ sqlsrv_context& henv_cp, _In_ sqlsrv_cont
#endif // !_WIN32

// time to free the access token, if not null
if (conn->azure_ad_access_token != NULL) {
if (conn->azure_ad_access_token) {
memset(conn->azure_ad_access_token->data, 0, conn->azure_ad_access_token->dataSize); // clear the memory
conn->azure_ad_access_token.reset();
}
Expand Down Expand Up @@ -980,11 +980,11 @@ void load_azure_key_vault(_Inout_ sqlsrv_conn* conn TSRMLS_DC)
throw core::CoreException();
}

CHECK_CUSTOM_ERROR(conn->ce_option.akv_id == NULL, conn, SQLSRV_ERROR_AKV_NAME_MISSING) {
CHECK_CUSTOM_ERROR(!conn->ce_option.akv_id, conn, SQLSRV_ERROR_AKV_NAME_MISSING) {
throw core::CoreException();
}

CHECK_CUSTOM_ERROR(conn->ce_option.akv_secret == NULL, conn, SQLSRV_ERROR_AKV_SECRET_MISSING) {
CHECK_CUSTOM_ERROR(!conn->ce_option.akv_secret, conn, SQLSRV_ERROR_AKV_SECRET_MISSING) {
throw core::CoreException();
}

Expand Down

0 comments on commit 71570ca

Please sign in to comment.