Report information
The Basics
Id:
45508
Status:
resolved
Priority:
Low/Low
Queue:
bind9-public
People
Owner:
Ray Bellis <ray@isc.org>
Requestors:
Ray Bellis <ray@isc.org>
Cc:
AdminCc:
BugTracker
Version Fixed:
9.12.0, 9.11.3, 9.10.7, 9.9.12
Version Found:
(no value)
Versions Affected:
(no value)
Versions Planned:
9.9.11, 9.10.7, 9.11.3
Priority:
P2 Normal
Severity:
S3 Low
CVSS Score:
(no value)
CVE ID:
(no value)
Component:
BIND Utilities
Area:
libdns
Attachments
libbind_dstcrash_test.tgz
Dates
Created:Wed, 05 Jul 2017 06:03:55 -0400
Updated:Fri, 11 Aug 2017 01:54:38 -0400
Closed:Fri, 11 Aug 2017 01:54:28 -0400

This bug tracker is no longer active.

Please go to our Gitlab to submit issues (both feature requests and bug reports) for active projects maintained by Internet Systems Consortium (ISC).

Due to security and confidentiality requirements, full access is limited to the primary maintainers.

History
  Wed, 05 Jul 2017 06:03:55 -0400 Ray Bellis <ray@isc.org> - Ticket created
From: ray@isc.org
To: bind9-public@isc.org
Subject: Crash if dst_lib_init() is called more than once
Date: Wed, 05 Jul 2017 10:03:55 +0000

In dig for iOS I want to be able to invoke the dig "main" function multiple times so that I can improve its error handling. However, when I try, I get a segfault in line 181 of lib/dns/dst_api.c, which calls: isc_mem_setdestroycheck(dst__memory_pool, ISC_FALSE); I've attached a standalone test program to reproduce this fault (the config.h therein is from my macOS build).
Subject: libbind_dstcrash_test.tgz
libbind_dstcrash_test.tgz

Message body not shown because it is not plain text.

Fri, 07 Jul 2017 10:12:28 -0400 Ray Bellis <ray@isc.org> - Status changed from 'new' to 'review'
Mon, 10 Jul 2017 16:39:31 -0400 Ray Bellis <ray@isc.org> - Dependency by #45481: iOS dig crashes when given an oversize label added
Wed, 12 Jul 2017 09:10:58 -0400 Francis Dupont <Francis_Dupont@isc.org> - Taken
Wed, 12 Jul 2017 09:46:00 -0400 The RT System itself - Subject changed from 'Crash if dst_lib_init() is called more than once' to 'Crash if dst_lib_init() is called more than once [ISC-Bugs #45508]'
Wed, 12 Jul 2017 09:48:30 -0400 Francis Dupont <Francis_Dupont@isc.org> - Given to Ray Bellis <ray@isc.org>
Fri, 21 Jul 2017 12:39:38 -0400 Brian Conry <bconry@isc.org> - Subject changed from 'Crash if dst_lib_init() is called more than once [ISC-Bugs #45508]' to 'Crash if dst_lib_init() is called more than once'
Mon, 31 Jul 2017 17:59:19 -0400 Vicky Risk <vicky@isc.org> - Versions Planned 9.9.11, 9.10.7, 9.11.3 added
Mon, 31 Jul 2017 17:59:27 -0400 Vicky Risk <vicky@isc.org> - Component Other changed to BIND Utilities
  Wed, 09 Aug 2017 19:17:01 -0400 Evan Hunt <Evan_Hunt@isc.org> - Correspondence added

SIGCHASE has now been removed from 9.12. I rebased this branch into rt45508b. That's the version that should be merged to master; rt45508 can be merged to 9.11 and earlier. I'm not sure where things stand with Francis's review, but these changes look okay to me.
  Fri, 11 Aug 2017 01:54:27 -0400 Evan Hunt <Evan_Hunt@isc.org> - Correspondence added

4677. [cleanup] Split up the main function in dig to better support the iOS app version. [RT #45508] 9.12.0, 9.11.3, 9.10.7, 9.9.12
Fri, 11 Aug 2017 01:54:28 -0400 Evan Hunt <Evan_Hunt@isc.org> - Status changed from 'review' to 'resolved'
Fri, 11 Aug 2017 01:54:38 -0400 Evan Hunt <Evan_Hunt@isc.org> - Version Fixed 9.12.0, 9.11.3, 9.10.7, 9.9.12 added