From 6208f47dcbf68ff1e751b0b526bb643f0da867a6 Mon Sep 17 00:00:00 2001 From: Pierre Ossman Date: Mon, 30 Dec 2019 10:26:12 +0100 Subject: [PATCH] Remove unneeded memory checks new throws an exception on allocation errors rather than return NULL. --- common/rfb/CSecurityTLS.cxx | 4 ---- 1 file changed, 4 deletions(-) diff --git a/common/rfb/CSecurityTLS.cxx b/common/rfb/CSecurityTLS.cxx index c1a00212..5c303a37 100644 --- a/common/rfb/CSecurityTLS.cxx +++ b/common/rfb/CSecurityTLS.cxx @@ -396,8 +396,6 @@ void CSecurityTLS::checkSession() vlog.debug("%s", info.data); certinfo = new char[len]; - if (certinfo == NULL) - throw AuthFailureException("Out of memory"); snprintf(certinfo, len, "This certificate has been signed by an unknown " "authority:\n\n%s\n\nDo you want to save it and " @@ -422,8 +420,6 @@ void CSecurityTLS::checkSession() // Save cert out_buf = new char[out_size]; - if (out_buf == NULL) - throw AuthFailureException("Out of memory"); if (gnutls_x509_crt_export(crt, GNUTLS_X509_FMT_PEM, out_buf, &out_size) < 0) throw AuthFailureException("certificate issuer unknown, and certificate " -- 2.39.5