diff options
| author | Martin Willi <martin@revosec.ch> | 2015-03-23 10:54:24 +0100 |
|---|---|---|
| committer | Martin Willi <martin@revosec.ch> | 2015-03-23 17:54:02 +0100 |
| commit | bace1d647971e8300c5abdb1e950adf3756ec328 (patch) | |
| tree | a0ccd1429564ad37130c64bb377c1f5de6bb92d0 /src/libtls/tls_server.c | |
| parent | 4909612c3bcfb951a3f8ecccf433756b450f4e8e (diff) | |
| download | strongswan-bace1d647971e8300c5abdb1e950adf3756ec328.tar.bz2 strongswan-bace1d647971e8300c5abdb1e950adf3756ec328.tar.xz | |
diffie-hellman: Use bool instead of status_t as get_shared_secret() return value
While such a change is not unproblematic, keeping status_t makes the API
inconsistent once we introduce return values for the public value operations.
Diffstat (limited to 'src/libtls/tls_server.c')
| -rw-r--r-- | src/libtls/tls_server.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/libtls/tls_server.c b/src/libtls/tls_server.c index b6e706d23..df5d00ab5 100644 --- a/src/libtls/tls_server.c +++ b/src/libtls/tls_server.c @@ -495,7 +495,7 @@ static status_t process_key_exchange_dhe(private_tls_server_t *this, pub = chunk_skip(pub, 1); } this->dh->set_other_public_value(this->dh, pub); - if (this->dh->get_shared_secret(this->dh, &premaster) != SUCCESS) + if (!this->dh->get_shared_secret(this->dh, &premaster)) { DBG1(DBG_TLS, "calculating premaster from DH failed"); this->alert->add(this->alert, TLS_FATAL, TLS_INTERNAL_ERROR); |