[asterisk-dev] [Code Review] 4528: dns: Add SRV recording parsing, sorting/weighting, and unit tests

[Joshua Colp]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4528/
-----------------------------------------------------------

(Updated March 26, 2015, 6:18 p.m.)


Status
------

This change has been discarded.


Review request for Asterisk Developers.


Repository: Asterisk


Description
-------

This change adds the following:

1. As SRV records are added to a result the information is parsed and stored away in additional storage in the record. The SRV API can then be used to return this information.
2. Before invoking the DNS query callback the list of records on the result are sorted based on priority and weight.
3. Unit tests have been added which verify the record parsing, sorting, and weighting. There are also some off nominal which cover the cases when an invalid/corrupt record is received.


Diffs
-----

  /trunk/tests/test_dns_srv.c PRE-CREATION 
  /trunk/main/dns_srv.c 433370 
  /trunk/main/dns_core.c 433370 
  /trunk/include/asterisk/dns_internal.h 433370 

Diff: https://reviewboard.asterisk.org/r/4528/diff/


Testing
-------

Executed unit tests and confirmed they pass.


Thanks,

Joshua Colp

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150326/21694335/attachment-0001.html>