Hi Robert,
Yes, I notice now - the reason I thought you hadn't checked it in is
that the first part of the change is subtly different from the patch you
sent me. I show the part that is different below. Maybe after you sent
it to me you've tweaked it a little.
Graeme.
@@ -257,6 +257,10 @@ netsnmp_row_merge_helper_handler(netsnmp
if (0 != rm_status->count)
netsnmp_assert(saved_requests[i] == request);
saved_requests[i] = request;
+ if (saved_requests[i]->prev) {
+ netsnmp_assert((i>0) && saved_requests[i]->prev ==
saved_requests[i-1]);
+ saved_requests[i]->prev = NULL;
+ }
}
saved_requests[i] = NULL;
> -----Original Message-----
> From: Robert Story [mailto:[EMAIL PROTECTED]
> Sent: 10 October 2006 17:58
> Cc: [email protected]; Graeme Wilson
> Subject: Re: add multiple rows with 1 set - crash
>
> On Tue, 10 Oct 2006 12:33:11 -0400 Robert wrote:
> RS> On Mon, 9 Oct 2006 15:10:44 +0100 Graeme wrote:
> RS> GW> Hi Robert,
> RS> GW>
> RS> GW> I can confirm that the patch you provided to
> row_merge.c fixes
> RS> GW> the problem of adding 2 rows to a table with 1 set.
> RS> GW> I notice that this hasn't yet been merged into the
> main net-snmp
> RS> GW> code yet. Do you plan on adding this in or were you
> waiting for
> RS> GW> a final ok from me?
> RS>
> RS> I was waiting for confirmation both to know it works, and as a
> RS> reminder. ;-) Will check it in today...
>
> Actually, it appears I checked it into main on 9/5.. don't
> know why you wouldn't see it..
>
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders
