Changeset 5522 for pjproject/trunk/pjsip/src/pjsua2/endpoint.cpp

Timestamp:

Jan 11, 2017 11:13:57 AM (8 years ago)

Author:

ming

Message:

Re #1960: Use boolean field to indicate whether the info is empty (instead of relying on bzero and memcmp)

File:

• pjproject/trunk/pjsip/src/pjsua2/endpoint.cpp (modified) (4 diffs)

pjproject/trunk/pjsip/src/pjsua2/endpoint.cpp

@@ -46 +46 @@
 
 TlsInfo::TlsInfo()
-{
-    pj_bzero(this, sizeof(TlsInfo));
+        : empty(true)
+{
 }
 
 bool TlsInfo::isEmpty() const
 {
-    TlsInfo dummy;
-
-    pj_bzero(&dummy, sizeof(dummy));
-    return ((pj_memcmp(this, &dummy, sizeof(dummy)) == 0)? true: false);
+    return empty;
 }
 
@@ -66 +63 @@
     unsigned verif_msg_cnt;
     
+    empty       = false;
     established = PJ2BOOL(ssock_info->established);
     protocol    = ssock_info->proto;
@@ -90 +88 @@
 }
 
+SslCertInfo::SslCertInfo()
+        : empty(true)
+{
+}
+
 bool SslCertInfo::isEmpty() const
 {
-    SslCertInfo dummy;
-
-    pj_bzero(&dummy, sizeof(dummy));
-    return ((pj_memcmp(this, &dummy, sizeof(dummy)) == 0)? true: false);
+    return empty;
 }
 
 void SslCertInfo::fromPj(const pj_ssl_cert_info &info)
 {
+    empty       = false;
     version     = info.version;
     pj_memcpy(serialNo, info.serial_no, sizeof(info.serial_no));
@@ -638 +639 @@
     prm.state = state;
     prm.lastError = info ? info->status : PJ_SUCCESS;
-    pj_bzero(&prm.tlsInfo, sizeof(TlsInfo));
 
 #if defined(PJSIP_HAS_TLS_TRANSPORT) && PJSIP_HAS_TLS_TRANSPORT!=0