Kristina Pfaff-Harris <[EMAIL PROTECTED]> wrote: > I've noticed a couple of minor things too. I'm thinking that the problems > I was having with SNMP and Simultaneous-Use killing the server earlier > were actually in part due to this: > > $sess_id = hex $ARGV[4]; > > On Ascend session ID's and, I think, others, this causes an "Integer > overflow in hexadecimal number" error, which seems to lead to the SNMP > errors. From what I can tell, if the snmpget call works on plain old > $ARGV[4], then we shouldn't need to even try to hex it. Isn't that right?
Hmm... probably. In fact, I'm not even sure why that is there in the first place. > In other minor "fixes," also, I made a config variable $cmmty_string > towards the top of the file, and changed "public" to $cmmty_string > throughout the script. I made $finger a config variable, just in case, and > put that at the top of the script as well. I also put $cmmty_string in '' > at one point where it's an argument to the external snmpwalk, in case your > community string has characters that lend themselves to being enclosed. That makes sense. > Patch attached covers the above for checkrad.pl.in from freeradius 0.8.1 > dist version. (I'm using Pine. In case silly Pine wraps lines, try this > URL: Applied, thanks. Alan DeKok. - List info/subscribe/unsubscribe? See http://www.freeradius.org/list/users.html