On 07/20/2013 06:49 PM, Magnus Fromreide wrote: > On Fri, 2013-07-19 at 16:09 +0800, zhuyj wrote: >> Would you like to give me some advice to make this patch better? > I could do a partial commit of a corrected patch that covers the indices > in snmpTargetAddrTable right now, but that leaves snmpTargetParamsTable > still buggy and then I have to make it handle NUL characters in the > TagLists and the Param selector. Hi, How to reproduce this? Would you like to tell me the steps? Maybe I can do something about this.
Zhuyj >> On 07/16/2013 04:49 PM, zhuyj wrote: >>> Hi, Magnus >>> >>> Based on your patch, I made a new patch. This patch depends on >>> net-snmp5.7.2. >>> I made the following modifications compared with your patch: >>> 1.add strdup2 to replace strdup; >>> 2.modify addToList function; >>> 3.change some NUL-terminated strings to counted-strings, but not all; >>> >>> The patch is in the attachment. Please check it. >>> >>> Best Regards! >>> zhuyj >>> On 07/16/2013 02:14 PM, Magnus Fromreide wrote: >>>> On Tue, 2013-07-16 at 10:27 +0800, zhuyj wrote: >>>>> Thanks for your help! >>>>> I agree with you. It is a big task to replace all the NUL-terminated >>>>> string with counted-string. >>>>> Now I have a patch that can fix this. Would you like to merge this >>>>> patch in master of net-snmp? >>>> If you send out the patch then I will look at it. >>>> >>>> /MF >>>> >>>> >>>> > > ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Net-snmp-coders mailing list Net-snmp-coders@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/net-snmp-coders