diff options
| author | Szabolcs Nagy <nsz@port70.net> | 2014-06-05 22:52:40 +0200 |
|---|---|---|
| committer | Szabolcs Nagy <nsz@port70.net> | 2014-06-05 23:06:37 +0200 |
| commit | 2abb70c302efe46dfd8fd9e1d64fa00f1376f428 (patch) | |
| tree | 869f116a88556e51d9a4bc365bf47ad2532d3f8b /src/network/res_mkquery.c | |
| parent | b3d9e0b94ea73c68ef4169ec82c898ce59a4e30a (diff) | |
| download | musl-2abb70c302efe46dfd8fd9e1d64fa00f1376f428.tar.gz musl-2abb70c302efe46dfd8fd9e1d64fa00f1376f428.tar.bz2 musl-2abb70c302efe46dfd8fd9e1d64fa00f1376f428.tar.xz musl-2abb70c302efe46dfd8fd9e1d64fa00f1376f428.zip | |
fix the domain name length limit checks
A domain name is at most 255 bytes long (RFC 1035), but the string
representation is two bytes smaller so the strlen maximum is 253.
Diffstat (limited to 'src/network/res_mkquery.c')
| -rw-r--r-- | src/network/res_mkquery.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/network/res_mkquery.c b/src/network/res_mkquery.c index f7e4e9c6..7c49709e 100644 --- a/src/network/res_mkquery.c +++ b/src/network/res_mkquery.c @@ -10,9 +10,9 @@ int __res_mkquery(int op, const char *dname, int class, int type, int id, i, j; unsigned char q[280]; struct timespec ts; - size_t l = strnlen(dname, 256); + size_t l = strnlen(dname, 254); - if (l-1>=254 || buflen<18+l || op>15u || class>255u || type>255u) + if (l-1>=253 || buflen<18+l || op>15u || class>255u || type>255u) return -1; /* Construct query template - ID will be filled later */ |