Opened 5 years ago

Closed 3 years ago

#3956 closed defect (fixed)

Cryptic configuration error when specifying invalid address pool boundaries

Reported by: marcin Owned by: UnAssigned
Priority: medium Milestone: Kea1.2
Component: remote-management Version: git
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: Medium
Sub-Project: DHCP Feature Depending on Ticket:
Estimated Difficulty: 0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description (last modified by tomek)

I've got a really cryptic configuration error with the following subnet configuration:

    "subnet": "2001:db8:2::/64",
       "pools": [
        {
             "pool": "2001:db8:2::100 - 2001:db8:1::200"
        } ]

And here is the error message:

2015-07-16 09:58:59.237 ERROR [kea-dhcp6.dhcp6/11378] DHCP6_INIT_FAIL failed to initialize Kea server: configuration error using file '../etc/kea/kea.conf': Upper boundary is smaller than lower boundary.

which either lacks information that it is the address boundary, or the line number.

We should probably also verify it for DHCPv4.

Subtickets

Change History (11)

comment:1 Changed 5 years ago by hschempf

  • Milestone changed from Kea-proposed to Kea1.0

per team meeting on Aug 5, move to 1.0 as low

comment:2 Changed 4 years ago by fdupont

It is a common issue (both DHCPv4 and DHCPv6, individual pool or pool list): Maker routines are not supposed to fail so the exception they raise escapes from the context. BTW I am afraid it is a generic issue, the first step should be to document syntax error exceptions so they can be caught in parsers.

comment:3 Changed 4 years ago by hschempf

  • Milestone changed from Kea1.0 to DHCP Outstanding Tasks

comment:4 Changed 4 years ago by tomek

  • Milestone changed from DHCP Outstanding Tasks to Outstanding Tasks

Milestone renamed

comment:5 Changed 3 years ago by tomek

  • Description modified (diff)
  • Milestone changed from Outstanding Tasks to Kea1.2

comment:6 Changed 3 years ago by tomek

  • Component changed from dhcp6 to configuration

comment:7 Changed 3 years ago by tomek

  • Component changed from configuration to remote-management

comment:8 Changed 3 years ago by fdupont

A BadValue exception is raised in Pool::Pool[46] constructors. The parser fails to add the location so I suggest to wait for the new parser (which ticket BTW?).

comment:9 Changed 3 years ago by fdupont

Addressed in #5097?

comment:10 Changed 3 years ago by fdupont

  • Owner set to UnAssigned
  • Status changed from new to reviewing

Now part of #5097 in src/bin/dhcp*/tests/config_parser_unittest.cc badPools.

comment:11 Changed 3 years ago by fdupont

  • Resolution set to fixed
  • Status changed from reviewing to closed

#5097 was merged: closing.

Note: See TracTickets for help on using tickets.