Re: [PATCH v2 5/5] Put database into country subdirectory

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> On Apr 30, 2018, at 10:54 AM, Philip Prindeville <philipp@xxxxxxxxxxxxxxxxxxxxx> wrote:
> 
> 
> 
>> On Apr 30, 2018, at 1:43 AM, Jan Engelhardt <jengelh@xxxxxxx> wrote:
>> 
>> On Monday 2018-04-30 02:06, Philip Prindeville wrote:
>> 
>>> 	/* Use simple integer vector files */
>>> 	if (nfproto == NFPROTO_IPV6) {
>>> -		snprintf(buf, sizeof(buf), GEOIP_DB_DIR "/%s.iv6", code);
>>> +		snprintf(buf, sizeof(buf), GEOIP_DB_DIR "/cc/%s.iv6", code);
>>> 	} else {
>>> -		snprintf(buf, sizeof(buf), GEOIP_DB_DIR "/%s.iv4", code);
>>> +		snprintf(buf, sizeof(buf), GEOIP_DB_DIR "/cc/%s.iv4", code);
>>> 	}
>> 
>> Why would this 5th patch be necessary? The integer vector
>> format did not change, did it?
> 
> 
> I’m working (in the next round of patches) to add ISP support as well, and don’t want them both in the same directories so I’ll be using subdirectories…
> 
> 


I think I answered the above pending question and followed up with a v3 set which had more functionality last month.

I’m wondering if there are changes I can make to advance this patch set or is it not going to proceed?

Thanks,

-Philip




[Index of Archives]     [Linux Netfilter Development]     [Linux Kernel Networking Development]     [Netem]     [Berkeley Packet Filter]     [Linux Kernel Development]     [Advanced Routing & Traffice Control]     [Bugtraq]

  Powered by Linux