意外的結果

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地址或網絡掩碼。