On Thu, Nov 25, 2010 at 10:25 AM, Fulko Hew <[email protected]> wrote:
> I was just reading some of the documentation for Net-SNMP and
I think I discovered another error...
In the following code snippet from the end of the file:
http://www.net-snmp.org/tutorial/tutorial-4/toolkit/mib_module/ustScalarSet.c
a) I don't think there should be a 'break' right after the case statement.
b) The comment about 'treat the rest the same as FREE make me think
there is a missing 'default:' immediately preceeding the 'case FREE:'
if not, it begs the question "the rest of what?".
/* Treat the rest the same as FREE */
case FREE:
break;
/* Release any resources that have been allocated */
if (new_string) {
free(new_string);
new_string = NULL;
}
if (old_string) {
free(old_string);
old_string = NULL;
}
break;
------------------------------------------------------------------------------
Increase Visibility of Your 3D Game App & Earn a Chance To Win $500!
Tap into the largest installed PC base & get more eyes on your game by
optimizing for Intel(R) Graphics Technology. Get started today with the
Intel(R) Software Partner Program. Five $500 cash prizes are up for grabs.
http://p.sf.net/sfu/intelisp-dev2dev
_______________________________________________
Net-snmp-users mailing list
[email protected]
Please see the following page to unsubscribe or change other options:
https://lists.sourceforge.net/lists/listinfo/net-snmp-users
