TSIG verify failure at DNS backend side

Bug #1466300 reported by Liang Rong
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Designate
Fix Released
High
sonu

Bug Description

Create a pool and configure the pool with a managed DNS backend (BIND9 in my case), and also create a TSIG key to identify the pool.

Next create zones in this pool. The zone transfer from MiniDNS to BIND9 fails because of TSIG verify failure at BIND9.

This issue is likely to be caused by the TSIG MAC generation in _handle_axfr function in mdns/handler.py. In this function, the request.request_mac should be replaced with request.mac. After I made this change, the TSIG verification is passed at BIND9 and zone transfer is completed successfully.

Tags: mdns
Revision history for this message
Kiall Mac Innes (kiall) wrote :

Looking into this, I'm not sure it warrants Private Security as it sounds like it's not leaking info / privilege escalation etc - but let's leave it private for now until we know for sure.

Revision history for this message
Liang Rong (csulrong) wrote :

Sorry, the Security option is mistakenly checked on.

Tim Simmons (timsim)
information type: Private Security → Public
Changed in designate:
importance: Undecided → High
status: New → Triaged
sonu (sonu-bhumca11)
Changed in designate:
assignee: nobody → sonu (sonu-bhumca11)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to designate (master)

Fix proposed to branch: master
Review: https://review.openstack.org/223524

Changed in designate:
status: Triaged → In Progress
Kiall Mac Innes (kiall)
Changed in designate:
milestone: none → liberty-rc1
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to designate (master)

Reviewed: https://review.openstack.org/223524
Committed: https://git.openstack.org/cgit/openstack/designate/commit/?id=da1fdf792ac926044ac79780a776a8caf669ca94
Submitter: Jenkins
Branch: master

commit da1fdf792ac926044ac79780a776a8caf669ca94
Author: sonu.kumar <email address hidden>
Date: Tue Sep 15 16:31:50 2015 +0530

    Replace request.request_mac with request.mac

    This patch replaces request.request_mac with request.mac
    to provide TSIG verification.

    Change-Id: Ide4f48dc3b4de7a919cc41abebf9843d0ba77763
    Closes-Bug: #1466300

Changed in designate:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in designate:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in designate:
milestone: liberty-rc1 → 1.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.