diff options
| author | wolfbeast <mcwerewolf@wolfbeast.com> | 2020-01-09 21:39:28 +0100 |
|---|---|---|
| committer | wolfbeast <mcwerewolf@wolfbeast.com> | 2020-01-09 21:39:28 +0100 |
| commit | 8198126c39ff8752972c32bb877af5df5fb05c8e (patch) | |
| tree | a9a876cffc2e9b8cd140aff9b0e4af8649443988 /security/manager/ssl/TransportSecurityInfo.cpp | |
| parent | d429ac8a6035cd278ed1dc32599118e04ca0d2b6 (diff) | |
| download | UXP-8198126c39ff8752972c32bb877af5df5fb05c8e.tar UXP-8198126c39ff8752972c32bb877af5df5fb05c8e.tar.gz UXP-8198126c39ff8752972c32bb877af5df5fb05c8e.tar.lz UXP-8198126c39ff8752972c32bb877af5df5fb05c8e.tar.xz UXP-8198126c39ff8752972c32bb877af5df5fb05c8e.zip | |
Be more consistent about decoding IP addresses in PSM.
Diffstat (limited to 'security/manager/ssl/TransportSecurityInfo.cpp')
| -rw-r--r-- | security/manager/ssl/TransportSecurityInfo.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/security/manager/ssl/TransportSecurityInfo.cpp b/security/manager/ssl/TransportSecurityInfo.cpp index 3c7023302..3f4bf4a90 100644 --- a/security/manager/ssl/TransportSecurityInfo.cpp +++ b/security/manager/ssl/TransportSecurityInfo.cpp @@ -8,6 +8,7 @@ #include "PSMRunnable.h" #include "mozilla/Casting.h" +#include "mozilla/net/DNS.h" #include "nsComponentManagerUtils.h" #include "nsIArray.h" #include "nsICertOverrideService.h" @@ -681,8 +682,10 @@ GetSubjectAltNames(CERTCertificate* nssCert, nsString& allNames) case certIPAddress: { - char buf[INET6_ADDRSTRLEN]; + // According to DNS.h, this includes space for the null-terminator + char buf[net::kNetAddrMaxCStrBufSize] = {0}; PRNetAddr addr; + memset(&addr, 0, sizeof(addr)); if (current->name.other.len == 4) { addr.inet.family = PR_AF_INET; memcpy(&addr.inet.ip, current->name.other.data, current->name.other.len); |