|
 |
a03026 |
From 62cb936cb7ad5f219715515ae7d32dd281a5aa1f Mon Sep 17 00:00:00 2001
|
|
|
a03026 |
From: Simon Kelley <simon@thekelleys.org.uk>
|
|
|
a03026 |
Date: Tue, 26 Sep 2017 22:00:11 +0100
|
|
|
a03026 |
Subject: Security fix, CVE-2017-14491, DNS heap buffer overflow.
|
|
|
a03026 |
|
|
|
a03026 |
Further fix to 0549c73b7ea6b22a3c49beb4d432f185a81efcbc
|
|
|
a03026 |
Handles case when RR name is not a pointer to the question,
|
|
|
a03026 |
only occurs for some auth-mode replies, therefore not
|
|
|
a03026 |
detected by fuzzing (?)
|
|
|
a03026 |
---
|
|
|
a03026 |
src/rfc1035.c | 27 +++++++++++++++------------
|
|
|
a03026 |
1 file changed, 15 insertions(+), 12 deletions(-)
|
|
|
a03026 |
|
|
|
a03026 |
diff --git a/src/rfc1035.c b/src/rfc1035.c
|
|
|
a03026 |
index 27af023..56ab88b 100644
|
|
|
a03026 |
--- a/src/rfc1035.c
|
|
|
a03026 |
+++ b/src/rfc1035.c
|
|
|
a03026 |
@@ -1086,32 +1086,35 @@ int add_resource_record(struct dns_header *header, char *limit, int *truncp, int
|
|
|
a03026 |
|
|
|
a03026 |
va_start(ap, format); /* make ap point to 1st unamed argument */
|
|
|
a03026 |
|
|
|
a03026 |
- /* nameoffset (1 or 2) + type (2) + class (2) + ttl (4) + 0 (2) */
|
|
|
a03026 |
- CHECK_LIMIT(12);
|
|
|
a03026 |
-
|
|
|
a03026 |
if (nameoffset > 0)
|
|
|
a03026 |
{
|
|
|
a03026 |
+ CHECK_LIMIT(2);
|
|
|
a03026 |
PUTSHORT(nameoffset | 0xc000, p);
|
|
|
a03026 |
}
|
|
|
a03026 |
else
|
|
|
a03026 |
{
|
|
|
a03026 |
char *name = va_arg(ap, char *);
|
|
|
a03026 |
- if (name)
|
|
|
a03026 |
- p = do_rfc1035_name(p, name, limit);
|
|
|
a03026 |
- if (!p)
|
|
|
a03026 |
- {
|
|
|
a03026 |
- va_end(ap);
|
|
|
a03026 |
- goto truncated;
|
|
|
a03026 |
- }
|
|
|
a03026 |
-
|
|
|
a03026 |
+ if (name && !(p = do_rfc1035_name(p, name, limit)))
|
|
|
a03026 |
+ {
|
|
|
a03026 |
+ va_end(ap);
|
|
|
a03026 |
+ goto truncated;
|
|
|
a03026 |
+ }
|
|
|
a03026 |
+
|
|
|
a03026 |
if (nameoffset < 0)
|
|
|
a03026 |
{
|
|
|
a03026 |
+ CHECK_LIMIT(2);
|
|
|
a03026 |
PUTSHORT(-nameoffset | 0xc000, p);
|
|
|
a03026 |
}
|
|
|
a03026 |
else
|
|
|
a03026 |
- *p++ = 0;
|
|
|
a03026 |
+ {
|
|
|
a03026 |
+ CHECK_LIMIT(1);
|
|
|
a03026 |
+ *p++ = 0;
|
|
|
a03026 |
+ }
|
|
|
a03026 |
}
|
|
|
a03026 |
|
|
|
a03026 |
+ /* type (2) + class (2) + ttl (4) + rdlen (2) */
|
|
|
a03026 |
+ CHECK_LIMIT(10);
|
|
|
a03026 |
+
|
|
|
a03026 |
PUTSHORT(type, p);
|
|
|
a03026 |
PUTSHORT(class, p);
|
|
|
a03026 |
PUTLONG(ttl, p); /* TTL */
|
|
|
a03026 |
--
|
|
|
a03026 |
2.7.4
|
|
|
a03026 |
|