解析主机名时防止异常

时间:2017-02-03 08:51:39

标签: c# sockets dns socketexception

我试图创建一个扫描网络以查找ARP请求并列出所有现有网络设备的应用。目前,我使用SharpPcapPacketDoNet

根据给定的IP解析主机名时,在解析"未知"时,我会得到SocketException。主办。所以我把它放在try / catch中。由于我认为这是不好的风格(忽略例外),我正在寻找不同的解决方案。

以下是一些代码:

// Button for scanning the network
private void btnStartScanningForClients_Click(object sender, RoutedEventArgs e)
{
    // Check for correct interface
    // [...]

    // Start scanning process
    if (!this.netWorkItOut.Startet)
    {
        // Dis-/Enable visual controls
        // [...]

        // Start scanning
        var index = this.cbNetworkInterface.SelectedIndex
        this.netWorkItOut.StartDevice(index);
        this.netWorkItOut.Scanner.StartScanningNetwork(resolveHostnames);
    }
}

这是控制对象,它保存扫描程序,处理事件,接受数据包并将它们放入队列

public void StartDevice(int deviceIndex)
{
    this.Startet = true;
    // [...]
    this.Device = WinPcapDeviceList.Instance[deviceIndex];

    // Activate Scanner
    this.Scanner = new Scanner(this.DeviceInfo);

    // Subscribe Events
    // [...]

    this.Device.Open(DeviceMode.Promiscuous, 1);
    this.Device.Filter = "(arp || ip || ip6)";

    this.Device.OnPacketArrival += device_OnPacketArrival;
    this.Device.StartCapture();
}

private void device_OnPacketArrival(object sender, CaptureEventArgs e)
{
    //PacketDoNet
    Packet packet;

    try
    { packet = Packet.ParsePacket(LinkLayers.Ethernet, e.Packet.Data); }
    catch (Exception)
    { return;  }

    if (packet is EthernetPacket)
    {
        var arp = ARPPacket.GetEncapsulated(packet);

        if (arp != null)
        {
            if (this.Scanner.Started)
            {
                lock (this.Scanner.PacketQueueARP)
                {
                    this.Scanner.PacketQueueARP.Add(arp);
                }
            }
        }
    }
}

这是Controlling对象和Scanner类。 scanner类用于处理ARP请求并解析主机名

public void StartScanningNetwork(bool resolveHostnames)
{
    // [...]
    this.ResolveHostnames = resolveHostnames;

    // start worker to listen for ARP packets
    this.workerARP = new Thread(WorkerARP);
    this.workerARP.Name = "Scanner thread (ARP)";
    this.workerARP.Start();

    this.Started = true;
}

private void WorkerARP()
{
    List<IPAddress> processedIps = new List<IPAddress>();

    // copy packets from storage queue to thread queue for processing
    while (Started)
    {
        // [...]

        if (this.threadQueueARP.Count > 0)
        {
            foreach (var packet in this.threadQueueARP)
            {
                // [...]

                if (!processedIps.Contains(ip))
                {
                    // [...]

                    if (this.ResolveHostnames)
                    {
                        var resolveHostnamesTask = Task.Factory.StartNew(ResolveHostnamesWorker, ip);
                    }
                }
                // [...]
            }

            // [...]
        }
        // [...]
    }

}

private void ResolveHostnamesWorker(object data)
{
    if (data is IPAddress)
    {
        var ip = (IPAddress)data;
        var hostname = "";

        try
        {
            hostname = Dns.GetHostEntry(ip).HostName;
        }
        catch { }

        // Raise Event for hostname resolved
    }
}

关于hostname = Dns.GetHostEntry(ip).HostName

行的全部内容

那么:在通过Dns.GetHostEntry()解析HostEntry时,如何避免使用try / catch?如果没有已知的主机,是否有一个仅返回null的函数?

提前致谢!

2 个答案:

答案 0 :(得分:3)

由于https://github.com/Microsoft/referencesource/blob/master/System/net/System/Net/DNS.cs下的班级Dns的代码可用,因此您只能使用相关部分。 (当然,您必须检查MIT许可证是否可以在您的项目中使用)

实现在方法InternalGetHostByAddress中,抛出异常。在那里你可以返回一个值(bool,enum ...),以便在查询成功时提供信息。

答案 1 :(得分:1)

据我所知,没有像TryGetHostName()这样的方法不会抛出异常。

但在我看来,根据您的预期捕捉异常是明显的。因此,您应该限制捕获您期望的异常:

private void ResolveHostnamesWorker(object data)
{
    if (data is IPAddress)
    {
        var ip = (IPAddress)data;
        var hostname = "";

        try
        {
            hostname = Dns.GetHostEntry(ip).HostName;
        }
        catch(SocketException socketException)
        {
            // maybe limit handling based on data in socketException and
            // call throw; to rethrow exception if not the expected one
        }

    // Raise Event for hostname resolved
}

}