Here's a fun little snippet:
Code:
for (int i = 0; i < (int)hostAddresses.Length; i++)
{
IPAddress pAddress = hostAddresses[i];
}
Not sure what you were trying to achieve, but I don't recommend doing this again. .FirstOrDefault() should work fine, unless you're trying to get only the last address in which case there's a better way to do that too.
The main area of interest seems to be the .UpdateTable call, which is raised as an event every 3 seconds or so. Thing is, your 'XOF' doesn't do what it says on the tin, or anything like it. Blacklisted and suspicious addresses are added to their respective string collections but beyond that your program doesn't appear to do anything meaningful with blocked addresses, they just sit there. I'm also looking at the IP table data grid you have on your Form object -- nothing seems to get added anywhere in your code except for a single call to .AddRange in the form initializer and the table ends up getting cleared every time .UpdateTable executes.
Also, in your call to BlockAddress, you initialize two objects and a string that are never assigned.
I don't know if you named the variables yourself, but if you did and my decompiler isn't just filling in missing debug data, "strs," "strs2," "str1," and "str3" are all terrible names for variables.
Edit: Nothing ever gets blacklisted because all that happens in your program is basically the following:
Code:
for remoteCon in connections
someList.add(remoteCon, 1)
if someList[remoteCon] > 40 then blockAddress
You may notice than 1 is never > 40
, what is this coded in?
