意外的结果

Question

在我们的组织，我们收到每日黑名单（更大，因为这仅仅是一个片断）的格式如下：

172.44.12.0

198.168.1.5

10.10。 0.0

192.168.78.6

192.168.22.22

111.111.0.0

222.222.0.0

12.12.12.12

当运行该程序的代码编译后我得到：

我使用C++在Linux/Unix环境。

到目前为止，我只是吐出来确保我的格式正确。

该文件的名称是blacklist.txt，其中包含目前上面列出的IP。我只使用cout来确保我的变量被正确定义。

#include <iostream> 
#include <vector> 
#include <fstream> 
#include <string> 
#include <netinet/in.h> 
#include <stdint.h> 
#include <arpa/inet.h> 

using namespace std; 

bool is_match(std::string &hay_stack, std::string &srcip) { 
    in_addr_t _ip = inet_addr(hay_stack.c_str()); 
    in_addr_t _IP = inet_addr(srcip.c_str()); 
    _ip = ntohl(_ip); 
    _IP = ntohl(_IP); 
    uint32_t mask=(_ip & 0x00ffffff == 0) ? 0xff000000 : 
    (_ip & 0x0000ffff == 0 ? 0xffff0000 : 0); 
    return ((_ip & mask) == (_IP & mask)); 
} 

int main() 
{ 
    vector<std::string> lines; 
    lines.reserve(5000); //Assuming that the file to read can have max 5K lines 

    string fileName("blacklist.txt"); 

    ifstream file; 
    file.open(fileName.c_str()); 

    if(!file.is_open()) 
    { 
     cerr<<"Error opening file : "<<fileName.c_str()<<endl; 
     return -1; 
    } 

    //Read the lines and store it in the vector 
    string line; 
    while(getline(file,line)) 
    { 
     lines.push_back(line); 
    } 

    file.close(); 

    //Dump all the lines in output 
    for(unsigned int i = 0; i < lines.size(); i++) 
    { 
     string h = lines[i]; 
     string mi = "10.10.10.10"; 
     cout<<is_match(h,mi)<<endl; 
    } 

    return 0; 
} 

我期待的输出为10.10.10.10（某种这里主机的子网）10.10.0.0（和某种这里子网掩码）

Answer 1

这是你的问题是：

uint32_t mask=(_ip & 0x00ffffff == 0) ? 0xff000000 : 
(_ip & 0x0000ffff == 0 ? 0xffff0000 : 0); 
return ((_ip & mask) == (_IP & mask)); 

如果_ip是形式x.0.0.0，它只有在_IP， 比较X和如果_ip是形式xy0.0，它只在_IP中比较x和y， 这很好。

但是，如果_ip不是这两种格式之一，则将掩码设置为0 < - 这是问题所在。 （_IP & 0），结果始终为0。 这意味着您始终在a.b.c.d，c！= 0或d！= 0的地址上返回true。

而是使默认掩码等于0xffffffff以检查完整匹配。

但事实证明，这不是大问题。最大的问题是，==比&更高的运算符优先级，让你的代码实际上是这样工作：

uint32_t mask=(_ip & (0x00ffffff == 0)) ? 0xff000000 : 
    (_ip & (0x0000ffff == 0) ? 0xffff0000 : 0); 
return ((_ip & mask) == (_IP & mask)); 

其结果是，你将永远的面具得到了0。你需要应用parens来解决这个问题。

所以在最后，你的代码应该改变看起来像这样：

uint32_t mask=((_ip & 0x00ffffff) == 0) ? 0xff000000 : 
    ((_ip & 0x0000ffff) == 0 ? 0xffff0000 : 0xffffffff); 
return ((_ip & mask) == (_IP & mask)); 

Answer 2

回应隐含的问题，“为什么我的程序没有按照我所期望的方式工作吗？“

我期待的输出为10.10.10.10（某种这里主机的子网）10.10.0.0（和某种这里子网掩码）

我不知道你为什么正在期待这一点。您的代码（如果该文件顺利打开）只中有一个print语句：

cout<<is_match(h,mi)<<endl; 

功能is_match总是返回一个布尔值，要么true或false。打印时，它总是分别为1或0。程序中没有任何代码可以打印IP地址或网络掩码。