SourceBans

According to the plugin code, a menu should've been shown to the admin who issued the ban asking them to pick a reason, but I don't think it was because a log message appeared saying that "Verify Insert Query Failed: Data too long for column 'reason' at row 1". That lead me to believe that a reason was never received from the admin.

I think the problem has to do with the decl for the reason string. decl doesn't guarantee what is going to be in the string when you access it and the test that happens later is looking for a \0 at [0] to determine if there is a reason.

In CommandBan

if(args >= 3)
{
     GetCmdArg(3, reason, sizeof(reason));
}

should be

if(args >= 3)
{
     GetCmdArg(3, reason, sizeof(reason));
}
else
{
     reason[0] = '\0';
}

This fix will also need to be applied to the unban, addban and banip commands.

I don't see any problems with this in unban, addban and banip. The reason is already set to \0.

Oh okay. I was just guessing ;)

This should fix it.

The attached plugin also has the admin unban bug fix.

Fix tested and working.

Fixed in r170.