Hi Imre - Thanks for the debdiff! The code changes look fine and passed
our build checks. I have a couple small formatting suggestions to follow
our normal security update style:

1) Since you recreated the patch based upon changes suggested in the
advisory, the patch should follow the DEP-3 patch tagging guidelines:

http://dep.debian.net/deps/dep3/

You really just need a "Description: " tag followed by some descriptive
text, along with an "Origin: upstream, http://doc.powerdns.com/powerdns-
advisory-2012-01.html" tag.

2) By using the DEP-3 guidelines, you would then drop the URL from the
changelog text and the patch description line would look something like
this:

    - debian/patches/CVE-2012-0206: Short, but meaningful, description
here. Based on upstream patch.

Note that there is no need to provide a debdiff for the Natty release,
as I will do a fake sync from the Debian update (we don't have a delta
on that specific package version).

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/918588

Title:
  PowerDNS Authoritative Server can be caused to generate a traffic loop
  CVE-2012-0206

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/pdns/+bug/918588/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to