Andrew Geissler | 82c905d | 2020-04-13 13:39:40 -0500 | [diff] [blame^] | 1 | From 0fcc0f210f3a9310a1963de640b384ce866410fd Mon Sep 17 00:00:00 2001 |
| 2 | From: Nate Karstens <nate.karstens@garmin.com> |
| 3 | Date: Wed, 9 Aug 2017 09:16:58 -0500 |
| 4 | Subject: [PATCH 08/11] Mark deleted interfaces as being changed |
| 5 | |
| 6 | Netlink notification handling ignores messages for deleted links, |
| 7 | RTM_DELLINK. It does handle RTM_GETLINK. According to libnl docu- |
| 8 | mentation (http://www.infradead.org/~tgr/libnl/doc/route.html) |
| 9 | RTM_DELLINK can be sent by the kernel, but RTM_GETLINK cannot. |
| 10 | There was likely a mixup in the original implementation, so this |
| 11 | change replaces handling for RTM_GETLINK with RTM_DELLINK. |
| 12 | |
| 13 | Testing and Verification Instructions: |
| 14 | 1. Use ip-link to add and remove a VLAN interface and verify |
| 15 | that mDNSResponder handles the deleted link. |
| 16 | |
| 17 | Upstream-Status: Submitted [dts@apple.com] |
| 18 | |
| 19 | Signed-off-by: Nate Karstens <nate.karstens@garmin.com> |
| 20 | --- |
| 21 | mDNSPosix/mDNSPosix.c | 2 +- |
| 22 | 1 file changed, 1 insertion(+), 1 deletion(-) |
| 23 | |
| 24 | diff --git a/mDNSPosix/mDNSPosix.c b/mDNSPosix/mDNSPosix.c |
| 25 | index 798ab10..a8a57df 100644 |
| 26 | --- a/mDNSPosix/mDNSPosix.c |
| 27 | +++ b/mDNSPosix/mDNSPosix.c |
| 28 | @@ -1163,7 +1163,7 @@ mDNSlocal void ProcessRoutingNotification(int sd, GenLinkedList *change |
| 29 | #endif |
| 30 | |
| 31 | // Process the NetLink message |
| 32 | - if (pNLMsg->nlmsg_type == RTM_GETLINK || pNLMsg->nlmsg_type == RTM_NEWLINK) |
| 33 | + if (pNLMsg->nlmsg_type == RTM_DELLINK || pNLMsg->nlmsg_type == RTM_NEWLINK) |
| 34 | AddInterfaceIndexToList(changedInterfaces, ((struct ifinfomsg*) NLMSG_DATA(pNLMsg))->ifi_index); |
| 35 | else if (pNLMsg->nlmsg_type == RTM_DELADDR || pNLMsg->nlmsg_type == RTM_NEWADDR) |
| 36 | AddInterfaceIndexToList(changedInterfaces, ((struct ifaddrmsg*) NLMSG_DATA(pNLMsg))->ifa_index); |
| 37 | -- |
| 38 | 2.17.1 |
| 39 | |