dns: Fix crash when invoking cancel in DNS recurring unit test.
authorJoshua Colp <jcolp@digium.com>
Thu, 2 Jul 2015 11:54:51 +0000 (08:54 -0300)
committerJoshua Colp <jcolp@digium.com>
Thu, 2 Jul 2015 11:54:51 +0000 (08:54 -0300)
commitf18436642b90487a53b8818ef853205620960c29
treea34a48b553b65e25020216826cae170ba29e795f
parentc12ace3ab31bd61606252f5a3721ce720d2ad209
dns: Fix crash when invoking cancel in DNS recurring unit test.

The recurring unit test expects the user data on a DNS query
created as a result of a recurring DNS query to be the recurring
structure itself. This is true, mostly. When invoking the user
provided callback this user data is changed to the user provided
data. This presents a race condition where the data may or may
not point to the recurring data.

This change simplifies the callback of the user provided callback
by creating a new query and populating it with the expected values.
This leaves the recurring DNS query alone and fixes the race
condition. This is more in line with how the API should be used
overall.

ASTERISK-25222 #close

Change-Id: I10fb6deec025dff097157e7ec17e6e4921778478
main/dns_recurring.c