diff options
| author | Adam Tkac <atkac@redhat.com> | 2012-03-05 14:22:52 +0100 |
|---|---|---|
| committer | Adam Tkac <atkac@redhat.com> | 2012-03-05 14:22:52 +0100 |
| commit | 6cd938d8ced0ffb1b7b331995653fa84bf9f7205 (patch) | |
| tree | 7d871f621d2fe4478b59d76f34a6df3ef6abc77e /bind99-coverity.patch | |
| parent | dabddcf2c781b3d3ecab1e1fc27e0553a8f3eb26 (diff) | |
| download | bind-6cd938d8ced0ffb1b7b331995653fa84bf9f7205.tar.gz bind-6cd938d8ced0ffb1b7b331995653fa84bf9f7205.tar.xz bind-6cd938d8ced0ffb1b7b331995653fa84bf9f7205.zip | |
Include some fixes...
- load dynamic DBs later (and update dyndb patch)
- fix memory leak in named during processing of rndc command
Signed-off-by: Adam Tkac <atkac@redhat.com>
Diffstat (limited to 'bind99-coverity.patch')
| -rw-r--r-- | bind99-coverity.patch | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/bind99-coverity.patch b/bind99-coverity.patch new file mode 100644 index 0000000..ac8d5d1 --- /dev/null +++ b/bind99-coverity.patch @@ -0,0 +1,21 @@ +diff -up bind-9.9.0/bin/named/controlconf.c.coverity bind-9.9.0/bin/named/controlconf.c +--- bind-9.9.0/bin/named/controlconf.c.coverity 2012-03-05 13:57:37.665539706 +0100 ++++ bind-9.9.0/bin/named/controlconf.c 2012-03-05 14:15:15.981420790 +0100 +@@ -373,14 +373,9 @@ control_recvmessage(isc_task_t *task, is + if (result == ISC_R_SUCCESS) + break; + isc_mem_put(listener->mctx, secret.rstart, REGION_SIZE(secret)); +- if (result == ISCCC_R_BADAUTH) { +- /* +- * For some reason, request is non-NULL when +- * isccc_cc_fromwire returns ISCCC_R_BADAUTH. +- */ +- if (request != NULL) +- isccc_sexpr_free(&request); +- } else { ++ /* request can be non-NULL even in failure case */ ++ isccc_sexpr_free(&request); ++ if (result != ISCCC_R_BADAUTH) { + log_invalid(&conn->ccmsg, result); + goto cleanup; + } |