logo       
Google Custom Search
    AddThis Social Bookmark Button
-->

netSnmpHostsTable.c: msg#00097

Subject: netSnmpHostsTable.c
I noticed a problem when looking at the netSnmpHostsTable_handler()
function.
If multiple varbinds for different instances are specified in the set
request, the
positioning of the initialization of the suffix and suffix_len variables
will cause problems
for the commitInfo and undoInfo structure processing.
These variables are currently initialized outside the for loop in the
current CVS code
base.

The following lines of code need to be moved into the "for (request = ... "
loop.
    /** column and row index encoded portion */
    suffix = requests->requestvb->name + reginfo->rootoid_len + 1;
    suffix_len = requests->requestvb->name_length - (reginfo->rootoid_len +
1);

Also, "requests" should be changed to "request"

Barb


-------------------------------------------------------
This SF.Net email sponsored by: ApacheCon 2003,
16-19 November in Las Vegas. Learn firsthand the latest
developments in Apache, PHP, Perl, XML, Java, MySQL,
WebDAV, and more! http://www.apachecon.com/


<Prev in Thread] Current Thread [Next in Thread>