Skip to content

Commit

Permalink
bsock: removed an unneeded ASSERT macro
Browse files Browse the repository at this point in the history
- tls_conn is set at the beginning of a tls session, in case of a failure
  the session is already aborted before, thus the tls_conn pointer is always
  set at this point wich makes the ASSERT redundant
  • Loading branch information
franku committed Sep 17, 2018
1 parent 7923f31 commit 8bb1a43
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions core/src/lib/bsock.cc
Expand Up @@ -393,8 +393,6 @@ bool BareosSocket::TwoWayAuthenticate(JobControlRecord *jcr,

bool BareosSocket::DoTlsHandshakeAsAServer(ConfigurationParser *config, JobControlRecord *jcr)
{
ASSERT(!tls_conn);

TlsResource *tls_resource = reinterpret_cast<TlsResource *>(config->GetNextRes(config->r_own_, nullptr));

if (!ParameterizeAndInitTlsConnectionAsAServer(config)) { return false; }
Expand Down

0 comments on commit 8bb1a43

Please sign in to comment.