Opened 9 years ago

Closed 9 years ago

#244 closed defect (fixed)

Loadzone cannot handle missing name and ttl

Reported by: stephen Owned by: jreed
Priority: medium Milestone:
Component: loadzone Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: 0.0 Add Hours to Ticket:
Total Hours: Internal?: no

Description

b10-loadzone does not load the following zone file correctly:

$ORIGIN example.com.
$TTL 60
@    IN SOA   ns1.example.com. hostmaster.example.com. (1 43200 900 1814400 7200)
     IN NS    ns1.example.com.
     IN MX 10 mail.example.com.
ns1  IN A     192.168.1.102
mail IN A     192.168.1.103

The NS and MX records are created with a name of IN.example.com. (instead of example.com.)

Subtickets

Change History (6)

comment:1 Changed 9 years ago by jreed

Even with tingting-loadzone branch I get:

sqlite> select * from zones ;
1|example.com.|IN|0
sqlite> select * from records ;
1|1|example.com.|com.example.|60|SOA||ns1.example.com. hostmaster.example.com. 1 43200 900 1814400 7200
2|1|IN.example.com.|com.example.IN.|60|NS||ns1.example.com.
3|1|IN.example.com.|com.example.IN.|60|MX||10 mail.example.com.
4|1|ns1.example.com.|com.example.ns1.|60|A||192.168.1.102
5|1|mail.example.com.|com.example.mail.|60|A||192.168.1.103

Note not all have .IN. as part of name. Only problem for the records that don't have a label but do have a class.

comment:2 follow-up: Changed 9 years ago by shentingting

I updated the code. The newest version has already fixed this bug.

comment:3 in reply to: ↑ 2 Changed 9 years ago by shentingting

Replying to shentingting:

I updated the code. The newest version has already fixed this bug.

By the way, the code is in branches/tingting-loadzone branch, version 2261

comment:4 Changed 9 years ago by shentingting

The code has been committed in r2340 in trunk, please check.

comment:5 Changed 9 years ago by jreed

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

comment:6 Changed 9 years ago by shentingting

  • billable set to 0
  • Estimated Difficulty set to 0.0
  • Internal? unset
  • Resolution set to fixed
  • Status changed from reviewing to closed

OK. this code has been merged. so close this.

Note: See TracTickets for help on using tickets.