mirror of
https://github.com/irssi/irssi.git
synced 2025-02-02 15:08:01 -05:00
Kill do { ... } while (0); and replace it with goto's.
This commit is contained in:
parent
d501a54f4f
commit
0a6e66f8b7
@ -776,34 +776,32 @@ int irssi_ssl_handshake(GIOChannel *handle)
|
|||||||
|
|
||||||
ret = 1;
|
ret = 1;
|
||||||
|
|
||||||
do {
|
if (pinned_cert_fingerprint != NULL && pinned_cert_fingerprint[0] != '\0') {
|
||||||
if (pinned_cert_fingerprint != NULL && pinned_cert_fingerprint[0] != '\0') {
|
ret = g_ascii_strcasecmp(pinned_cert_fingerprint, tls->certificate_fingerprint) == 0;
|
||||||
ret = g_ascii_strcasecmp(pinned_cert_fingerprint, tls->certificate_fingerprint) == 0;
|
|
||||||
|
|
||||||
if (! ret) {
|
if (! ret) {
|
||||||
g_warning(" Pinned certificate mismatch");
|
g_warning(" Pinned certificate mismatch");
|
||||||
continue;
|
goto done;
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
if (pinned_pubkey_fingerprint != NULL && pinned_pubkey_fingerprint[0] != '\0') {
|
if (pinned_pubkey_fingerprint != NULL && pinned_pubkey_fingerprint[0] != '\0') {
|
||||||
ret = g_ascii_strcasecmp(pinned_pubkey_fingerprint, tls->public_key_fingerprint) == 0;
|
ret = g_ascii_strcasecmp(pinned_pubkey_fingerprint, tls->public_key_fingerprint) == 0;
|
||||||
|
|
||||||
if (! ret) {
|
if (! ret) {
|
||||||
g_warning(" Pinned public key mismatch");
|
g_warning(" Pinned public key mismatch");
|
||||||
continue;
|
goto done;
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
if (chan->verify) {
|
if (chan->verify) {
|
||||||
ret = irssi_ssl_verify(chan->ssl, chan->ctx, chan->server->connrec->address, chan->port, cert, chan->server, tls);
|
ret = irssi_ssl_verify(chan->ssl, chan->ctx, chan->server->connrec->address, chan->port, cert, chan->server, tls);
|
||||||
|
|
||||||
if (! ret) {
|
if (! ret) {
|
||||||
// irssi_ssl_verify emits a warning itself.
|
// irssi_ssl_verify emits a warning itself.
|
||||||
continue;
|
goto done;
|
||||||
}
|
|
||||||
}
|
}
|
||||||
} while (0);
|
}
|
||||||
|
|
||||||
done:
|
done:
|
||||||
tls_rec_free(tls);
|
tls_rec_free(tls);
|
||||||
|
Loading…
Reference in New Issue
Block a user