On Wed, 1 Feb 2006 15:23:25 -0500 Robert wrote:
RS> MR> changing the length to size_t fixed the crashes.:)
RS> 
RS> That makes no sense to me. The compiler ought to know the sizes of things
RS> and convert. There's no blind casting to confuse things...

But...

table_tdata.c:49: warning: passing argument 2 of 'build_oid' from incompatible
pointer type

Provides a clue to where the compiler has done a cast for us... And fixing
that warning via the use of a temporary variable does fix the crash.


RS> I hate the idea of making OID lengths size_t... The max value is 128, for
RS> pete's sake! Any unsigned char is big enough!

So, there is a real fix for the actual issue. The question is, should we go
ahead an make the length a size_t just because of the new mentality that 'any
counter/length should be size_t'?  I think my bias is pretty obvious, but I
figured I'd poll the rest of you..

-- 
NOTE: messages sent directly to me, instead of the lists, will be deleted
      unless they are requests for paid consulting services.

Robert Story; NET-SNMP Junkie
Support: <http://www.net-snmp.org/> <irc://irc.freenode.net/#net-snmp>
Archive: <http://sourceforge.net/mailarchive/forum.php?forum=net-snmp-coders>

You are lost in a twisty maze of little standards, all different. 


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to