]> granicus.if.org Git - curl/commit
x509asn1: Fix SAN IP address verification
authorMatthew Whitehead <matthew1001@gmail.com>
Mon, 15 Oct 2018 15:27:28 +0000 (16:27 +0100)
committerJay Satiro <raysatiro@yahoo.com>
Tue, 16 Oct 2018 07:52:47 +0000 (03:52 -0400)
commitdf54b14fb77bc7f62f31971ed8bb26ec24bf27d5
tree384f152db88c172b6b5f313a15a4f27c67e81688
parent03186b118784e067cccef7d469564d8fbb96725f
x509asn1: Fix SAN IP address verification

For IP addresses in the subject alternative name field, the length
of the IP address (and hence the number of bytes to perform a
memcmp on) is incorrectly calculated to be zero. The code previously
subtracted q from name.end. where in a successful case q = name.end
and therefore addrlen equalled 0. The change modifies the code to
subtract name.beg from name.end to calculate the length correctly.

The issue only affects libcurl with GSKit SSL, not other SSL backends.
The issue is not a security issue as IP verification would always fail.

Fixes #3102
Closes #3141
lib/x509asn1.c