]> git.ipfire.org Git - thirdparty/bind9.git/log
thirdparty/bind9.git
7 years agoFixes 52-dot
Witold Kręcicki [Wed, 7 Nov 2018 13:12:27 +0000 (13:12 +0000)] 
Fixes

7 years agoWiP 2
Witold Kręcicki [Sun, 4 Nov 2018 18:07:27 +0000 (18:07 +0000)] 
WiP 2

7 years agoWiP1
Witold Kręcicki [Fri, 2 Nov 2018 20:30:56 +0000 (20:30 +0000)] 
WiP1

7 years agoFreeBSD fixes - proper affinity setting, don't use SO_REUSEPORT
Witold Krecicki [Fri, 26 Oct 2018 12:41:48 +0000 (14:41 +0200)] 
FreeBSD fixes - proper affinity setting, don't use SO_REUSEPORT

7 years agoDon't doublelock fdlock on Solaris
Witold Krecicki [Fri, 26 Oct 2018 12:22:02 +0000 (14:22 +0200)] 
Don't doublelock fdlock on Solaris

7 years agoSynchronize Windows socket code with API changes.
Witold Kręcicki [Wed, 24 Oct 2018 10:52:37 +0000 (10:52 +0000)] 
Synchronize Windows socket code with API changes.

7 years agoFix fetchlimit test to allow softquota to be overruled; Remove 'verifying that named...
Witold Kręcicki [Thu, 18 Oct 2018 14:45:28 +0000 (14:45 +0000)] 
Fix fetchlimit test to allow softquota to be overruled; Remove 'verifying that named checks for conflicting listeners' test from runtime

7 years agoUse multiple worker queues (using isc_task_sendto()) in unix socket code
Witold Kręcicki [Thu, 18 Oct 2018 22:00:15 +0000 (22:00 +0000)] 
Use multiple worker queues (using isc_task_sendto()) in unix socket code

7 years agouse REUSEPORT only for NOLISTEN dispatchers
Witold Kręcicki [Thu, 11 Oct 2018 07:50:31 +0000 (07:50 +0000)] 
use REUSEPORT only for NOLISTEN dispatchers

7 years agoMake watching/unwatching sockets saner.
Witold Kręcicki [Thu, 4 Oct 2018 14:52:54 +0000 (14:52 +0000)] 
Make watching/unwatching sockets saner.

7 years agoUse the newly accepted FD in TCP code.
Witold Kręcicki [Thu, 4 Oct 2018 14:32:28 +0000 (14:32 +0000)] 
Use the newly accepted FD in TCP code.

7 years agoUse isc_refcount_t in socket reference counting.
Witold Kręcicki [Thu, 4 Oct 2018 12:30:04 +0000 (12:30 +0000)] 
Use isc_refcount_t in socket reference counting.

7 years agoMultiple network event loop threads support for Solarises /dev/poll.
Witold Krecicki [Thu, 4 Oct 2018 14:11:24 +0000 (16:11 +0200)] 
Multiple network event loop threads support for Solarises /dev/poll.

7 years agoMangle fdlocks a bit.
Witold Kręcicki [Wed, 3 Oct 2018 22:11:52 +0000 (22:11 +0000)] 
Mangle fdlocks a bit.

Mutexes are slower if they're in the same cache line. Since
fd's come in herds, and usually our listen sockets will have nearby
fd numbers, we mangle fdlocks so that the locks are further away.

7 years agoFix a race between socket closing and incoming event.
Witold Kręcicki [Wed, 3 Oct 2018 20:39:39 +0000 (20:39 +0000)] 
Fix a race between socket closing and incoming event.

7 years agoSet network threads CPU affintity.
Witold Kręcicki [Thu, 18 Oct 2018 20:47:23 +0000 (20:47 +0000)] 
Set network threads CPU affintity.

7 years agoAdd runtime detection of SO_REUSEPORT, use it instead of dup() if available.
Witold Kręcicki [Thu, 23 Aug 2018 07:42:30 +0000 (09:42 +0200)] 
Add runtime detection of SO_REUSEPORT, use it instead of dup() if available.

7 years agoUse the same number of UDP dispatchers as CPUs.
Witold Kręcicki [Wed, 22 Aug 2018 13:41:28 +0000 (15:41 +0200)] 
Use the same number of UDP dispatchers as CPUs.

7 years agoUse multiple network event loop threads with separate data structures.
Witold Krecicki [Wed, 3 Oct 2018 20:46:23 +0000 (22:46 +0200)] 
Use multiple network event loop threads with separate data structures.

7 years agoDo IO after event directly in the network thread, don't queue an event in a separate...
Witold Kręcicki [Tue, 21 Aug 2018 12:04:06 +0000 (14:04 +0200)] 
Do IO after event directly in the network thread, don't queue an event in a separate task.

7 years agoRemove fdwatch sockets, those are not used anywhere.
Witold Kręcicki [Mon, 20 Aug 2018 09:56:37 +0000 (11:56 +0200)] 
Remove fdwatch sockets, those are not used anywhere.

7 years agoMerge branch 'wpk-multiple-task-queues' into 'master'
Witold Krecicki [Tue, 6 Nov 2018 09:44:34 +0000 (04:44 -0500)] 
Merge branch 'wpk-multiple-task-queues' into 'master'

Multiple task queues

See merge request isc-projects/bind9!878

7 years agoCHANGES note
Witold Kręcicki [Tue, 6 Nov 2018 08:17:33 +0000 (08:17 +0000)] 
CHANGES note

7 years agoUse larger quantum for network tasks
Witold Kręcicki [Tue, 30 Oct 2018 15:07:25 +0000 (15:07 +0000)] 
Use larger quantum for network tasks

7 years agoUse a single wake_all_queues() function to wake all queues
Witold Kręcicki [Thu, 25 Oct 2018 15:01:25 +0000 (15:01 +0000)] 
Use a single wake_all_queues() function to wake all queues

7 years agoSwitch from privileged to un-privileged mode under lock
Witold Kręcicki [Thu, 25 Oct 2018 12:41:59 +0000 (12:41 +0000)] 
Switch from privileged to un-privileged mode under lock

7 years agoDocument isc_task_sendto properly, make sure that cpu we're sending to is always...
Witold Kręcicki [Thu, 25 Oct 2018 06:31:53 +0000 (06:31 +0000)] 
Document isc_task_sendto properly, make sure that cpu we're sending to is always sane

7 years agoUse proper memory ordering for tasks_running/tasks_ready
Witold Kręcicki [Thu, 25 Oct 2018 06:27:24 +0000 (06:27 +0000)] 
Use proper memory ordering for tasks_running/tasks_ready

7 years agoComment about taskmgr exclusive mode, fix a REQUIRE.
Witold Kręcicki [Tue, 23 Oct 2018 09:39:56 +0000 (09:39 +0000)] 
Comment about taskmgr exclusive mode, fix a REQUIRE.

7 years agoAllow slight over-quota in'checking lame server clients are dropped at the soft limit...
Witold Kręcicki [Tue, 23 Oct 2018 09:03:31 +0000 (09:03 +0000)] 
Allow slight over-quota in'checking lame server clients are dropped at the soft limit' test

7 years agoSaner exclusive task handling in taskmgr
Witold Kręcicki [Tue, 23 Oct 2018 08:47:44 +0000 (08:47 +0000)] 
Saner exclusive task handling in taskmgr

7 years agoFormatting
Witold Kręcicki [Tue, 23 Oct 2018 08:20:17 +0000 (08:20 +0000)] 
Formatting

7 years agoGet rid of isc_taskmgr_setmode, we only use it to set privileged mode
Witold Kręcicki [Mon, 22 Oct 2018 11:18:45 +0000 (11:18 +0000)] 
Get rid of isc_taskmgr_setmode, we only use it to set privileged mode

7 years agoMake sure all priority tasks are done before entering normal execution
Witold Kręcicki [Mon, 22 Oct 2018 10:57:05 +0000 (10:57 +0000)] 
Make sure all priority tasks are done before entering normal execution

7 years agoPost shutting down tasks always to manager 0
Witold Kręcicki [Mon, 22 Oct 2018 12:26:27 +0000 (12:26 +0000)] 
Post shutting down tasks always to manager 0

7 years agoSeparate structure for each thread/queue; 2-phase-locking for exclusive tasks
Witold Kręcicki [Mon, 22 Oct 2018 09:37:17 +0000 (09:37 +0000)] 
Separate structure for each thread/queue; 2-phase-locking for exclusive tasks

7 years agoAlways restart dispatchers on empty readyq
Witold Kręcicki [Fri, 19 Oct 2018 10:13:20 +0000 (10:13 +0000)] 
Always restart dispatchers on empty readyq

7 years agoTaskmgr shutdown fixes
Witold Kręcicki [Thu, 18 Oct 2018 18:16:25 +0000 (18:16 +0000)] 
Taskmgr shutdown fixes

7 years agoMultiple worker queues
Witold Kręcicki [Thu, 11 Oct 2018 13:39:04 +0000 (13:39 +0000)] 
Multiple worker queues

7 years agoisc_thread_setaffinity()
Witold Krecicki [Wed, 3 Oct 2018 21:20:22 +0000 (23:20 +0200)] 
isc_thread_setaffinity()

7 years agoMerge branch 'rndc-nta-preen' into 'master'
Evan Hunt [Tue, 6 Nov 2018 06:18:32 +0000 (01:18 -0500)] 
Merge branch 'rndc-nta-preen' into 'master'

Fixes for `rndc nta` user interface

See merge request isc-projects/bind9!845

7 years agoCHANGES, release note
Evan Hunt [Thu, 4 Oct 2018 20:39:48 +0000 (13:39 -0700)] 
CHANGES, release note

7 years agoFixes for `rndc nta` user interface
Tony Finch [Tue, 6 Dec 2016 14:32:47 +0000 (14:32 +0000)] 
Fixes for `rndc nta` user interface

Tell the user explicitly about their mistakes:

* Unknown options, e.g. -list instead of -dump
  or -delete instead of -remove.

* Unknown view names.

* Excess arguments.

Include the view name in `rndc nta -dump` output, for consistency with
the NTA add and remove actions.

When removing an NTA from all views, do not abort with an error if the
NTA was not found in one of the views.

7 years agoMerge branch '653-idnout-only-on-tty' into 'master'
Mark Andrews [Tue, 6 Nov 2018 05:58:22 +0000 (00:58 -0500)] 
Merge branch '653-idnout-only-on-tty' into 'master'

Enable idnout output only on tty, disable it when the stdout is not a tty

Closes #653

See merge request isc-projects/bind9!973

7 years agoAdd CHANGES entry for GL #653
Ondřej Surý [Tue, 6 Nov 2018 04:55:57 +0000 (11:55 +0700)] 
Add CHANGES entry for GL #653

7 years agoAdd release notes for IDN processing update
Ondřej Surý [Tue, 6 Nov 2018 04:53:31 +0000 (11:53 +0700)] 
Add release notes for IDN processing update

7 years agoUpdate the IDN documentation for nslookup
Ondřej Surý [Tue, 6 Nov 2018 04:46:25 +0000 (11:46 +0700)] 
Update the IDN documentation for nslookup

7 years agoChange the dig documentation
Ondřej Surý [Tue, 6 Nov 2018 04:45:13 +0000 (11:45 +0700)] 
Change the dig documentation

7 years agoUpdate the idna tests for the new non-tty defaults
Ondřej Surý [Sat, 3 Nov 2018 19:41:33 +0000 (02:41 +0700)] 
Update the idna tests for the new non-tty defaults

7 years agoEnable IDN processing (both idnin and idnout) only on tty, disable it when the stdout...
Ondřej Surý [Sat, 3 Nov 2018 19:20:41 +0000 (02:20 +0700)] 
Enable IDN processing (both idnin and idnout) only on tty, disable it when the stdout is not a tty

7 years agoMerge branch '601-build-option-with-python-should-default-to-yes' into 'master'
Mark Andrews [Tue, 6 Nov 2018 05:38:06 +0000 (00:38 -0500)] 
Merge branch '601-build-option-with-python-should-default-to-yes' into 'master'

Resolve "Build option --with-python should default to yes"

Closes #601

See merge request isc-projects/bind9!980

7 years agorequire python to be explicitly disabled
Mark Andrews [Tue, 6 Nov 2018 04:35:02 +0000 (15:35 +1100)] 
require python to be explicitly disabled

7 years agoMerge branch 'u/fanf2/deprecate-ip6-int' into 'master'
Mark Andrews [Tue, 6 Nov 2018 00:29:23 +0000 (19:29 -0500)] 
Merge branch 'u/fanf2/deprecate-ip6-int' into 'master'

Abolish ip6.int support in `dig` and `mdig`

See merge request isc-projects/bind9!969

7 years agoadd CHANGES note
Mark Andrews [Tue, 6 Nov 2018 00:19:03 +0000 (11:19 +1100)] 
add CHANGES note

7 years agoAbolish ip6.int support in `dig` and `mdig`
Tony Finch [Fri, 2 Nov 2018 14:23:01 +0000 (14:23 +0000)] 
Abolish ip6.int support in `dig` and `mdig`

7 years agoMerge branch '617-if-rrl-is-configured-the-require-server-cookie-yes-is-ignored'...
Mark Andrews [Mon, 5 Nov 2018 23:18:37 +0000 (18:18 -0500)] 
Merge branch '617-if-rrl-is-configured-the-require-server-cookie-yes-is-ignored' into 'master'

Resolve "If RRL is configured the "require-server-cookie yes;" is ignored."

Closes #617

See merge request isc-projects/bind9!974

7 years agoadd CHANGES note
Mark Andrews [Mon, 5 Nov 2018 05:01:40 +0000 (16:01 +1100)] 
add CHANGES note

7 years agotest require-server-cookie with rate-limit
Mark Andrews [Mon, 5 Nov 2018 04:55:19 +0000 (15:55 +1100)] 
test require-server-cookie with rate-limit

7 years agocheck requireservercookie even if rrl is configured
Mark Andrews [Mon, 5 Nov 2018 01:33:54 +0000 (12:33 +1100)] 
check requireservercookie even if rrl is configured

7 years agoMerge branch '634-unchecked-returns-in-resolver-c' into 'master'
Mark Andrews [Mon, 5 Nov 2018 23:03:05 +0000 (18:03 -0500)] 
Merge branch '634-unchecked-returns-in-resolver-c' into 'master'

Resolve "Unchecked returns in resolver.c"

Closes #634

See merge request isc-projects/bind9!929

7 years agotwo dns_name_dup calls were not checked
Mark Andrews [Mon, 29 Oct 2018 07:04:54 +0000 (18:04 +1100)] 
two dns_name_dup calls were not checked

7 years agoMerge branch '645-fix-missing-buffer-init' into 'master'
Ondřej Surý [Mon, 5 Nov 2018 20:05:48 +0000 (15:05 -0500)] 
Merge branch '645-fix-missing-buffer-init' into 'master'

dighost.c: after invalidating buffer we need to reinit it

Closes #645

See merge request isc-projects/bind9!976

7 years agodighost.c: after invalidating buffer we need to reinit it
Witold Kręcicki [Mon, 5 Nov 2018 18:59:44 +0000 (18:59 +0000)] 
dighost.c: after invalidating buffer we need to reinit it

7 years agoMerge branch '16-cleanup-finds-before-sending-final-query' into 'master'
Witold Krecicki [Mon, 5 Nov 2018 10:06:32 +0000 (05:06 -0500)] 
Merge branch '16-cleanup-finds-before-sending-final-query' into 'master'

Cleanup fctx->finds before sending 'final' query after qname minimization.

Closes #16

See merge request isc-projects/bind9!962

7 years agoCleanup fctx->finds before sending 'final' query after qname minimization.
Witold Kręcicki [Wed, 31 Oct 2018 11:23:57 +0000 (11:23 +0000)] 
Cleanup fctx->finds before sending 'final' query after qname minimization.

At the beginning of qname minimization we get fctx->finds filled with what's
in the cache at this point, in worst case root servers. After doing full
run querying for NSes at different levels we need to clean it and refill
it with proper values from cache.

7 years agoMerge branch 'fix-dnstap-test' into 'master'
Evan Hunt [Fri, 2 Nov 2018 18:35:54 +0000 (14:35 -0400)] 
Merge branch 'fix-dnstap-test' into 'master'

disable QNAME minimization in the dnstap test so packet counts will be correct

See merge request isc-projects/bind9!971

7 years agodisable QNAME minimization in the dnstap test so packet counts will be correct
Evan Hunt [Fri, 2 Nov 2018 18:34:27 +0000 (11:34 -0700)] 
disable QNAME minimization in the dnstap test so packet counts will be correct

7 years agoMerge branch 'wpk-qmin-intermittent-fixes' into 'master'
Evan Hunt [Fri, 2 Nov 2018 18:00:12 +0000 (14:00 -0400)] 
Merge branch 'wpk-qmin-intermittent-fixes' into 'master'

Avoid a race in qmin test

See merge request isc-projects/bind9!954

7 years agoAvoid a race in qmin test
Witold Kręcicki [Tue, 30 Oct 2018 14:51:33 +0000 (14:51 +0000)] 
Avoid a race in qmin test

7 years agoMerge branch 'u/fanf2/configure-backtrace' into 'master'
Mark Andrews [Fri, 2 Nov 2018 02:26:46 +0000 (22:26 -0400)] 
Merge branch 'u/fanf2/configure-backtrace' into 'master'

Correctly report backtrace support when `configure` finishes

See merge request isc-projects/bind9!967

7 years agoregen
Mark Andrews [Fri, 2 Nov 2018 02:09:04 +0000 (13:09 +1100)] 
regen

7 years agoCorrectly report backtrace support when `configure` finishes
Tony Finch [Thu, 1 Nov 2018 13:43:44 +0000 (13:43 +0000)] 
Correctly report backtrace support when `configure` finishes

7 years agoMerge branch '611-extend-serve-stale-tests' into 'master'
Michał Kępień [Wed, 31 Oct 2018 13:26:42 +0000 (09:26 -0400)] 
Merge branch '611-extend-serve-stale-tests' into 'master'

Extend serve-stale tests

Closes #611

See merge request isc-projects/bind9!963

7 years agoCheck serve-stale behavior with a cold cache
Michał Kępień [Mon, 22 Oct 2018 13:26:45 +0000 (15:26 +0200)] 
Check serve-stale behavior with a cold cache

Ensure that serve-stale works as expected when returning stale answers
is enabled, the authoritative server does not respond, and there is no
cached answer available.

7 years agoCheck TTL of stale answers
Michał Kępień [Mon, 22 Oct 2018 13:26:45 +0000 (15:26 +0200)] 
Check TTL of stale answers

Make sure that stale answers returned when the serve-stale feature is
enabled have a TTL matching the value of the stale-answer-ttl setting.

7 years agoMerge branch 'wpk-get-rid-of-socket-v-functions' into 'master'
Witold Krecicki [Wed, 31 Oct 2018 12:57:08 +0000 (08:57 -0400)] 
Merge branch 'wpk-get-rid-of-socket-v-functions' into 'master'

Get rid of socket *v functions

See merge request isc-projects/bind9!932

7 years ago[squash] Rework code preparing a HTTP response for sending in lib/isc/httpd.c
Michał Kępień [Wed, 31 Oct 2018 12:46:52 +0000 (13:46 +0100)] 
[squash] Rework code preparing a HTTP response for sending in lib/isc/httpd.c

7 years agoCHANGES entry
Witold Kręcicki [Tue, 30 Oct 2018 14:27:08 +0000 (14:27 +0000)] 
CHANGES entry

7 years agoRemove vector socket functions from Unix socket code and library headers
Witold Kręcicki [Fri, 26 Oct 2018 19:07:24 +0000 (19:07 +0000)] 
Remove vector socket functions from Unix socket code and library headers

Remove the following functions in order to simplify socket code:

  - isc_socket_recvv()
  - isc_socket_sendtov()
  - isc_socket_sendtov2()
  - isc_socket_sendv()

7 years agoRemove vector socket functions from Windows socket code
Witold Kręcicki [Mon, 29 Oct 2018 09:16:26 +0000 (09:16 +0000)] 
Remove vector socket functions from Windows socket code

Remove the following functions in order to simplify socket code:

  - isc_socket_recvv()
  - isc_socket_sendtov()
  - isc_socket_sendtov2()
  - isc_socket_sendv()

7 years agoDo not use vector socket functions in diagnostic tools
Witold Kręcicki [Mon, 29 Oct 2018 09:15:42 +0000 (09:15 +0000)] 
Do not use vector socket functions in diagnostic tools

Refactor diagnostic tools code to no longer use:

  - isc_socket_recvv()
  - isc_socket_sendtov2()
  - isc_socket_sendv()

as these functions will be removed shortly.

7 years agoDo not use vector socket functions in HTTP server code
Witold Kręcicki [Fri, 26 Oct 2018 19:07:41 +0000 (19:07 +0000)] 
Do not use vector socket functions in HTTP server code

Refactor code in lib/isc/httpd.c to no longer use isc_socket_sendv() as
this function will be removed shortly.

7 years agoMerge branch '649-resolver-test-failing' into 'master'
Mark Andrews [Wed, 31 Oct 2018 05:28:46 +0000 (01:28 -0400)] 
Merge branch '649-resolver-test-failing' into 'master'

Resolve "resolver test failing"

Closes #649

See merge request isc-projects/bind9!959

7 years agouse new packet for response; rename variable to request and response
Mark Andrews [Wed, 31 Oct 2018 05:14:43 +0000 (16:14 +1100)] 
use new packet for response; rename variable to request and response

7 years agoMerge branch 'wpk-fix-catz-test-race' into 'master'
Witold Krecicki [Tue, 30 Oct 2018 19:17:02 +0000 (15:17 -0400)] 
Merge branch 'wpk-fix-catz-test-race' into 'master'

catz test: wait for dom8.example to be transferred before checking its availability

See merge request isc-projects/bind9!946

7 years agoFix style in catz test
Witold Kręcicki [Tue, 30 Oct 2018 12:15:19 +0000 (13:15 +0100)] 
Fix style in catz test

7 years agocatz test: wait for dom8.example to be transferred before checking its availability
Witold Kręcicki [Tue, 30 Oct 2018 10:52:27 +0000 (11:52 +0100)] 
catz test: wait for dom8.example to be transferred before checking its availability

7 years agoMerge branch '647-config-h' into 'master'
Evan Hunt [Tue, 30 Oct 2018 17:51:13 +0000 (13:51 -0400)] 
Merge branch '647-config-h' into 'master'

remove config.h references

Closes #647

See merge request isc-projects/bind9!958

7 years agoremove config.h references
Evan Hunt [Tue, 30 Oct 2018 17:42:41 +0000 (10:42 -0700)] 
remove config.h references

7 years agoMerge branch 'silence-warning' into 'master'
Evan Hunt [Tue, 30 Oct 2018 15:07:53 +0000 (11:07 -0400)] 
Merge branch 'silence-warning' into 'master'

silence warning from missing print.h

See merge request isc-projects/bind9!955

7 years agosilence warning from missing print.h
Evan Hunt [Tue, 30 Oct 2018 15:06:34 +0000 (08:06 -0700)] 
silence warning from missing print.h

7 years agoMerge branch '84-destroy-task-first-when-destroying-rpzs' into 'master'
Ondřej Surý [Tue, 30 Oct 2018 13:02:02 +0000 (09:02 -0400)] 
Merge branch '84-destroy-task-first-when-destroying-rpzs' into 'master'

Destroy updater task first when destroying rpzs.

Closes #84

See merge request isc-projects/bind9!944

7 years agoCHANGES entry
Witold Kręcicki [Tue, 30 Oct 2018 07:14:57 +0000 (07:14 +0000)] 
CHANGES entry

7 years ago Destroy task first when destroying catzs.
Ondřej Surý [Tue, 30 Oct 2018 12:41:07 +0000 (13:41 +0100)] 
Destroy task first when destroying catzs.

    When freeing catzs structures we need to kill the updater task first.
    Otherwise we might race with the updater and there might be a crash
    on shutdown.

7 years agoDestroy task first when destroying rpzs.
Witold Kręcicki [Tue, 30 Oct 2018 07:09:58 +0000 (07:09 +0000)] 
Destroy task first when destroying rpzs.

When freeing rpzs structures we need to kill the updater task first.
Otherwise we might race with the updater and there might be a crash
on shutdown.

7 years agoMerge branch '644-fix-isc_buffer_copyregion-for-auto-reallocated-buffers' into 'master'
Michał Kępień [Tue, 30 Oct 2018 12:52:25 +0000 (08:52 -0400)] 
Merge branch '644-fix-isc_buffer_copyregion-for-auto-reallocated-buffers' into 'master'

Fix isc_buffer_copyregion() for auto-reallocated buffers

Closes #644

See merge request isc-projects/bind9!949

7 years agoAdd CHANGES entry
Michał Kępień [Tue, 30 Oct 2018 12:33:25 +0000 (13:33 +0100)] 
Add CHANGES entry

5072. [bug] Add unit tests for isc_buffer_copyregion() and fix its
behavior for auto-reallocated buffers. [GL #644]

7 years agoFix isc_buffer_copyregion() for auto-reallocated buffers
Michał Kępień [Tue, 30 Oct 2018 12:33:25 +0000 (13:33 +0100)] 
Fix isc_buffer_copyregion() for auto-reallocated buffers

While isc_buffer_copyregion() calls isc_buffer_reserve() to ensure the
target buffer will have enough available space to append the contents of
the source region to it, the variables used for subsequently checking
available space are not updated accordingly after that call.  This
prevents isc_buffer_copyregion() from working as expected for
auto-reallocated buffers: ISC_R_NOSPACE will be returned if enough space
is not already available in the target buffer before it is reallocated.
Fix by calling isc_buffer_used() and isc_buffer_availablelength()
directly instead of assigning their return values to local variables.

7 years agoAdd unit tests for isc_buffer_copyregion()
Michał Kępień [Tue, 30 Oct 2018 12:33:25 +0000 (13:33 +0100)] 
Add unit tests for isc_buffer_copyregion()

Add some basic checks for isc_buffer_copyregion() to ensure it behaves
as expected for both fixed-size buffers and buffers which can be
automatically reallocated.  Adjust the list of headers included by
lib/isc/tests/buffer_test.c so that it matches what that test program
really uses.

7 years agoMerge branch 'ondrej/cleanup-leftover-fatal' into 'master'
Ondřej Surý [Tue, 30 Oct 2018 11:56:51 +0000 (07:56 -0400)] 
Merge branch 'ondrej/cleanup-leftover-fatal' into 'master'

Remove double result check leftover

See merge request isc-projects/bind9!945