On 2006-11-12 15:18:10 +0100, Thomas Anders wrote:
> Magnus Fromreide wrote:
> > I am thus more inclined towards a patch that also changes the signature
> > of __scan_num_objid to
> >
> > static int __scan_num_objid _((char *, oid *, size_t *));
> >
> > to make the patch more correct.
>
> Agreed.
i just tested with this new patch on s390x, ppc64, ia64 and x86_64. The
perl testsuite finally passes on all those architectures.
+1
> > Additionally this looks like a general fault case that could be present
> > in more places so maybe a global search for size_t/int confusion might
> > be in place as well?
>
> I think it's overdue. I'm happy to attach a revised patch that changes all oid
> length variables/parameters consistently to size_t, matching the main library.
> I'm calling for votes to include this in 5.4.
+1 on finally fixing all the size_t/int problems.
darix
--
openSUSE - SUSE Linux is my linux
openSUSE is good for you
www.opensuse.org
-------------------------------------------------------------------------
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
