[Net-snmp-patches] [ net-snmp-Patches-1562100 ] FD is checked for -ve value before calling FD_ISSET()

[SourceForge.net <no...@so...>]

Patches item #1562100, was opened at 2006-09-20 04:08
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=312694&aid=1562100&group_id=12694

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: FD is checked for -ve value before calling FD_ISSET()

Initial Comment:
This patch is made against Net-snmp-5.2.2 version. We 
tested this patch on linux-i386 and mvl-ppc. Both 
snmpd and subagent worked fine with this patch.

In function _sess_read(), snmp_api.c, transport->sock 
is checked for negative value before calling FD_ISSET. 
If the transport->sock is less than 0, then _sess_read
() function logs the message "transport->sock got 
negative fd value" and will return 0 with this patch. 

Following were the observations and behavior of Agentx 
subagent without this patch.

We are using NET-SNMP-5.2.2 in subagent code.  We saw 
that subagent crashed when agent was crashed.  We 
found that, in _sess_read(), FD_ISSET() was crashed 
because transport->sock has the value -1 (0xFFFFFFFF).



----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=312694&aid=1562100&group_id=12694