diff options
| author | Vanya Sergeev <vsergeev@gmail.com> | 2013-06-08 15:49:45 -0700 |
|---|---|---|
| committer | Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> | 2013-07-19 17:35:56 +0200 |
| commit | 6e16f1768d75a3f5c367dcd6a5ab63f65fce5f82 (patch) | |
| tree | 80d1b58d9e200ca632e7418b79edd814026043de | |
| parent | 603af30d6992e94ac30a66b953264076f4f2fd71 (diff) | |
inet: fix unsafe access to _res.options in res_mkquery()
res_mkquery() takes out __resolv_lock to copy _res.options to function local
_res_options on line 4204, but later unsafely accesses _res.options without a
lock, instead of its local copy _res_options, on line 4221. Looks like a
period / underscore typo.
Signed-off-by: Vanya Sergeev <vsergeev@gmail.com>
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
| -rw-r--r-- | libc/inet/resolv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libc/inet/resolv.c b/libc/inet/resolv.c index ffd3fe8c4..8546b279a 100644 --- a/libc/inet/resolv.c +++ b/libc/inet/resolv.c @@ -4218,7 +4218,7 @@ int res_mkquery(int op, const char *dname, int class, int type, hp = (HEADER *) buf; hp->id = getpid() & 0xffff; hp->opcode = op; - hp->rd = (_res.options & RES_RECURSE) != 0U; + hp->rd = (_res_options & RES_RECURSE) != 0U; hp->rcode = NOERROR; cp = buf + HFIXEDSZ; |